Skip to content

Conversation

@codemzs
Copy link
Member

@codemzs codemzs commented Apr 11, 2019

fixes #3294

@codemzs codemzs requested review from sfilipi, wschin and zeahmed April 11, 2019 18:40
// Positive Precision: 0.50
// Positive Recall: 1.00
// LogLoss: 1.05
// LogLossReduction: -4.89
Copy link
Member

Choose a reason for hiding this comment

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

Why does it change the baseline?

Copy link
Member Author

@codemzs codemzs Apr 11, 2019

Choose a reason for hiding this comment

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

It didn't the documentation was stale. I verified this without the prior trainer change from the master branch (removed the change that changes label type to boolean) and below is the output:

Accuracy: 0.50
AUC: 0.50
F1 Score: 0.67
Negative Precision: 0.00
Negative Recall: 0.00
Positive Precision: 0.50
Positive Recall: 1.00
Log Loss: 1.05
Log Loss Reduction: -0.05
Entropy: 1.00

@codecov
Copy link

codecov bot commented Apr 11, 2019

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3295      +/-   ##
==========================================
- Coverage   72.63%   72.63%   -0.01%     
==========================================
  Files         807      807              
  Lines      145127   145127              
  Branches    16219    16219              
==========================================
- Hits       105414   105412       -2     
- Misses      35295    35298       +3     
+ Partials     4418     4417       -1
Flag Coverage Δ
#Debug 72.63% <ø> (-0.01%) ⬇️
#production 68.17% <ø> (ø) ⬆️
#test 88.94% <ø> (-0.02%) ⬇️
Impacted Files Coverage Δ
...est/Microsoft.ML.Predictor.Tests/TestPredictors.cs 63.76% <0%> (-0.42%) ⬇️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs 84.7% <0%> (-0.41%) ⬇️
...ML.Transforms/Text/StopWordsRemovingTransformer.cs 86.26% <0%> (+0.15%) ⬆️
src/Microsoft.ML.Maml/MAML.cs 26.21% <0%> (+1.45%) ⬆️

@codemzs codemzs requested a review from TomFinley April 11, 2019 20:27
Copy link
Contributor

@TomFinley TomFinley left a comment

Choose a reason for hiding this comment

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

Thank you @codemzs !

@codemzs codemzs merged commit 41ba426 into dotnet:master Apr 11, 2019
codemzs added a commit to codemzs/machinelearning that referenced this pull request Apr 11, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Mar 22, 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.

Prior Trainer sample throws an exception and the output comments do not match the actual output

4 participants