-
-
Notifications
You must be signed in to change notification settings - Fork 26.2k
FIX Bin training and validation data separately in GBDTs #13933
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIX Bin training and validation data separately in GBDTs #13933
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Maybe we can have a compound entry that summaries all the small changes to |
I added a regular whatsnew entry, we can make the compound entry when there's more to add |
ping @adrinjalali @glemaitre |
…n_train_val_separately
Should the |
Yes, good point |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Closes #13926
Instead of binning the whole data before the train/validation split, we now bin the training and validation data separately.
Not sure if that's worth a whatsnew entry, since it's all experimental still?