-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Pass fold index to cross validation metrics. #575
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
Pass fold index to cross validation metrics. #575
Conversation
@@ -162,6 +164,12 @@ internal static List<BinaryClassificationMetrics> FromMetrics(IHostEnvironment e | |||
/// </summary> | |||
public ConfusionMatrix ConfusionMatrix { get; private set; } | |||
|
|||
/// <summary> | |||
/// For cross validation gives fold number or standard deviation or average across all metrics. |
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.
fold number or standard deviation or average across all metrics [](start = 39, length = 63)
let's elaborate, because right now it feels unclear.
Maybe call it 'row tag', and then add a comment that
For cross-validation, this is equal to "Fold N" for per-fold metric rows, "Overall" for the average metrics and "STD" for standard deviation.
For non-CV scenarios, this is equal to null.
(whatever the actual strings are).
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.
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.
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.
Thanks for doing this.
RowTag in metrics
RowTag in metrics
Fixes #570