[R-package] Promote objective and init_score to top-level arguments in lightgbm()#4976
[R-package] Promote objective and init_score to top-level arguments in lightgbm()#4976StrikerRUS merged 12 commits intomicrosoft:masterfrom
lightgbm()#4976Conversation
jameslamb
left a comment
There was a problem hiding this comment.
Thanks for this. I support adding init_score and objective as keyword arguments in lightgbm(). Please see the requested changes.
Co-authored-by: James Lamb <jaylamb20@gmail.com>
|
Updated. |
jameslamb
left a comment
There was a problem hiding this comment.
Thanks very much for the tests! Please see my most recent requested changes.
jameslamb
left a comment
There was a problem hiding this comment.
Thanks very much! I think this is almost ready to merge, just a few more recommended changes.
|
Updated. |
jameslamb
left a comment
There was a problem hiding this comment.
thanks! Just one more suggestion and this can be merged.
|
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
ref #4968
This PR promotes
objectiveandinit_scoreto top-level parameters inlightgbm(). This is for 3 reasons:objectiveis probably one of the most common parameters to want to change. It's also accepted as a top-level parameter in thelgb.traininterface and in the scikit-learn interface.objectiveas a top-level parameter will allow an option"auto"in the future (as part of [R-package] User-friendly redesign forlightgbm()#4968) to adjust it automatically for binary and multi-class classification depending on the input.init_scoreas a top-level parameter will allow to take it as a column from the input data in the future (as part of [R-package] User-friendly redesign forlightgbm()#4968), even if it right now looks out of place.