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 documentation for LightGbm parameter #3695

Merged
merged 4 commits into from
May 15, 2019
Merged

Conversation

najeeb-kazmi
Copy link
Member

Fix #3687

Documentation for MaximumCategoricalSplitPoint.

Copy link
Contributor

@Ivanidzo4ka Ivanidzo4ka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@codecov
Copy link

codecov bot commented May 9, 2019

Codecov Report

Merging #3695 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3695      +/-   ##
==========================================
+ Coverage   72.77%   72.78%   +<.01%     
==========================================
  Files         808      808              
  Lines      145588   145588              
  Branches    16250    16250              
==========================================
+ Hits       105956   105965       +9     
+ Misses      35207    35201       -6     
+ Partials     4425     4422       -3
Flag Coverage Δ
#Debug 72.78% <ø> (ø) ⬆️
#production 68.28% <ø> (ø) ⬆️
#test 89.03% <ø> (ø) ⬆️
Impacted Files Coverage Δ
src/Microsoft.ML.LightGbm/LightGbmTrainerBase.cs 62.44% <ø> (ø) ⬆️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs 85.11% <0%> (+0.4%) ⬆️
src/Microsoft.ML.Transforms/Text/LdaTransform.cs 89.89% <0%> (+0.62%) ⬆️
src/Microsoft.ML.Maml/MAML.cs 26.21% <0%> (+1.45%) ⬆️

@najeeb-kazmi najeeb-kazmi self-assigned this May 14, 2019
codemzs and others added 3 commits May 14, 2019 21:46
* Brew install libomp 7.0.0 only.

* Brew install libomp 7.0.0 only.

* Brew install libomp 7.0.0 only.

* Brew install libomp 7.0.0 only.
Copy link
Member

@ganik ganik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@najeeb-kazmi najeeb-kazmi merged commit 0daa32c into dotnet:master May 15, 2019
@najeeb-kazmi najeeb-kazmi deleted the 3687 branch January 30, 2020 01:20
@ghost ghost locked as resolved and limited conversation to collaborators Mar 21, 2022
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.

LightGBM option MaximumCategoricalSplitPointCount has wrong doc
4 participants