Skip to content

Fix documentation for LightGbm parameter - #3695

Merged
najeeb-kazmi merged 4 commits into
dotnet:masterfrom
najeeb-kazmi:3687
May 15, 2019
Merged

Fix documentation for LightGbm parameter#3695
najeeb-kazmi merged 4 commits into
dotnet:masterfrom
najeeb-kazmi:3687

Conversation

@najeeb-kazmi

Copy link
Copy Markdown
Member

Fix #3687

Documentation for MaximumCategoricalSplitPoint.

@Ivanidzo4ka Ivanidzo4ka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:shipit:

@codecov

codecov Bot commented May 9, 2019

Copy link
Copy Markdown

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.
@najeeb-kazmi
najeeb-kazmi requested a review from shmoradims May 15, 2019 19:59

@ganik ganik left a comment

Copy link
Copy Markdown
Contributor

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

5 participants