Skip to content
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 the bug in bin with small values #2342

Merged
merged 3 commits into from
Aug 20, 2019
Merged

fix the bug in bin with small values #2342

merged 3 commits into from
Aug 20, 2019

Conversation

guolinke
Copy link
Collaborator

No description provided.

@guolinke
Copy link
Collaborator Author

ping @btrotta for #2299
#2299 will bring a check fail for all negative feature values, and produce additional bins for all positive feature values, and is different from previous bing results.

@guolinke guolinke merged commit 20f94c5 into master Aug 20, 2019
@btrotta
Copy link
Collaborator

btrotta commented Aug 20, 2019

@guolinke Sorry for the mistake. I'll update my current PR #2325 to match this behavior.

btrotta added a commit to btrotta/LightGBM that referenced this pull request Aug 20, 2019
btrotta added a commit to btrotta/LightGBM that referenced this pull request Aug 20, 2019
@StrikerRUS StrikerRUS deleted the guolinke-patch-1 branch August 20, 2019 12:21
btrotta added a commit to btrotta/LightGBM that referenced this pull request Sep 10, 2019
btrotta added a commit to btrotta/LightGBM that referenced this pull request Sep 10, 2019
guolinke pushed a commit that referenced this pull request Sep 28, 2019
* Fix bug where small values of max_bin cause crash.

* Revert "Fix bug where small values of max_bin cause crash."

This reverts commit fe5c8e2.

* Add functionality to force bin thresholds.

* Fix style issues.

* Use stable sort.

* Minor style and doc fixes.

* Add functionality to force bin thresholds.

* Fix style issues.

* Use stable sort.

* Minor style and doc fixes.

* Change binning behavior to be same as PR #2342.

* Add functionality to force bin thresholds.

* Fix style issues.

* Use stable sort.

* Minor style and doc fixes.

* Add functionality to force bin thresholds.

* Fix style issues.

* Use stable sort.

* Minor style and doc fixes.

* Change binning behavior to be same as PR #2342.

* Add functionality to force bin thresholds.

* Fix style issues.

* Minor style and doc fixes.

* Add functionality to force bin thresholds.

* Fix style issues.

* Minor style and doc fixes.

* Change binning behavior to be same as PR #2342.

* Add functionality to force bin thresholds.

* Fix style issues.

* Use stable sort.

* Minor style and doc fixes.

* Add functionality to force bin thresholds.

* Fix style issues.

* Use stable sort.

* Minor style and doc fixes.

* Change binning behavior to be same as PR #2342.

* Use different bin finding function for predefined bounds.

* Fix style issues.

* Minor refactoring, overload FindBinWithZeroAsOneBin.

* Fix style issues.

* Fix bug and add new test.

* Add warning when using categorical features with forced bins.

* Pass forced_upper_bounds by reference.

* Pass container types by const reference.

* Get categorical features using FeatureBinMapper.

* Fix bug for small max_bin.

* Move GetForcedBins to DatasetLoader.

* Find forced bins in dataset_loader.

* Minor fixes.
@lock lock bot locked as resolved and limited conversation to collaborators Mar 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants