-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Predefined bin thresholds #2325
Merged
Merged
Changes from all commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
fe5c8e2
Fix bug where small values of max_bin cause crash.
btrotta 439bcfd
Revert "Fix bug where small values of max_bin cause crash."
btrotta 34e72c8
Add functionality to force bin thresholds.
btrotta 5b21573
Fix style issues.
btrotta 2be599a
Use stable sort.
btrotta 6a098f0
Minor style and doc fixes.
btrotta 0cd4abc
Merge remote-tracking branch 'upstream/master'
btrotta 8f73636
Add functionality to force bin thresholds.
btrotta 6c2d048
Fix style issues.
btrotta feb861f
Use stable sort.
btrotta 873fa64
Minor style and doc fixes.
btrotta 050f57b
Merge branch 'force-bin' of https://github.com/btrotta/lightgbm into …
btrotta 4cd89e4
Change binning behavior to be same as PR #2342.
btrotta 698d9db
Merge remote-tracking branch 'upstream/master'
btrotta 9d22071
Add functionality to force bin thresholds.
btrotta 3178609
Fix style issues.
btrotta 934b305
Use stable sort.
btrotta dc45bd1
Minor style and doc fixes.
btrotta 018182c
Add functionality to force bin thresholds.
btrotta 7a4df51
Fix style issues.
btrotta 6095148
Use stable sort.
btrotta 8b57a56
Minor style and doc fixes.
btrotta de83a69
Change binning behavior to be same as PR #2342.
btrotta 01f18fd
Merge branch 'force-bin' of https://github.com/btrotta/lightgbm into …
btrotta 360eacf
Merge remote-tracking branch 'upstream/master'
btrotta c478775
Add functionality to force bin thresholds.
btrotta e3f1835
Fix style issues.
btrotta 2280c56
Minor style and doc fixes.
btrotta 76fa4cc
Add functionality to force bin thresholds.
btrotta 93d92eb
Fix style issues.
btrotta fec30a5
Minor style and doc fixes.
btrotta 503e7b4
Change binning behavior to be same as PR #2342.
btrotta eecb80c
Add functionality to force bin thresholds.
btrotta a02b3a3
Fix style issues.
btrotta cb12379
Use stable sort.
btrotta abe95d7
Minor style and doc fixes.
btrotta 7aed689
Add functionality to force bin thresholds.
btrotta 35ce38b
Fix style issues.
btrotta 28c0462
Use stable sort.
btrotta 23dbb29
Minor style and doc fixes.
btrotta 9ed04a3
Change binning behavior to be same as PR #2342.
btrotta 7cdc732
Fix merge conflict.
btrotta 51e93a9
Use different bin finding function for predefined bounds.
btrotta 4e3355a
Fix style issues.
btrotta 821b2ab
Minor refactoring, overload FindBinWithZeroAsOneBin.
btrotta 8a52444
Fix style issues.
btrotta c591e7b
Fix bug and add new test.
btrotta 9c767ae
Add warning when using categorical features with forced bins.
btrotta cf0afd4
Pass forced_upper_bounds by reference.
btrotta 25387ec
Pass container types by const reference.
btrotta cc249f0
Get categorical features using FeatureBinMapper.
btrotta 0e26e9f
Fix bug for small max_bin.
btrotta feeb163
Merge remote-tracking branch 'upstream/master'
btrotta 50ff73b
Fix merge conflicts.
btrotta b5752ec
Move GetForcedBins to DatasetLoader.
btrotta 58d86aa
Find forced bins in dataset_loader.
btrotta 3e81b94
Minor fixes.
btrotta File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
[ | ||
{ | ||
"feature": 0, | ||
"bin_upper_bound": [ 0.3, 0.35, 0.4 ] | ||
}, | ||
{ | ||
"feature": 1, | ||
"bin_upper_bound": [ -0.1, -0.15, -0.2 ] | ||
} | ||
] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
[ | ||
{ | ||
"feature": 0, | ||
"bin_upper_bound": [ 0.19, 0.39, 0.59, 0.79 ] | ||
} | ||
] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
CopyFeatureMapperFrom
,CreateValid
also need to copyforced_bin_bounds_
.