-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Cross Validation and TrainTest #212
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
Changes from 13 commits
3145971
f3e42ef
4ce6605
c39dfc7
6f273a3
3543ddb
d9aa28c
5676a27
e930904
857d5ab
ae376a0
363cad8
0ef9ce1
001fa53
3f53309
372e764
95f4e83
eaa5545
2769d9b
6371e06
1f520a2
3991971
3424775
542b845
072e61d
9d6fb01
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -469,6 +469,35 @@ | |
| "ILearningPipelineLoader" | ||
| ] | ||
| }, | ||
| { | ||
| "Name": "Data.TransformModelArrayConverter", | ||
| "Desc": "Create and array variable", | ||
| "FriendlyName": null, | ||
| "ShortName": null, | ||
| "Inputs": [ | ||
| { | ||
| "Name": "TransformModel", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "TransformModel" | ||
| }, | ||
| "Desc": "The models", | ||
| "Required": true, | ||
| "SortOrder": 1.0, | ||
| "IsNullable": false | ||
| } | ||
| ], | ||
| "Outputs": [ | ||
| { | ||
| "Name": "OutputModel", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "TransformModel" | ||
| }, | ||
| "Desc": "The model array" | ||
| } | ||
| ] | ||
| }, | ||
| { | ||
| "Name": "Models.AnomalyDetectionEvaluator", | ||
| "Desc": "Evaluates an anomaly detection scored dataset.", | ||
|
|
@@ -1238,6 +1267,116 @@ | |
| "IEvaluatorOutput" | ||
| ] | ||
| }, | ||
| { | ||
| "Name": "Models.CrossValidationResultsCombiner", | ||
| "Desc": "Combine the metric data views returned from cross validation.", | ||
| "FriendlyName": null, | ||
| "ShortName": null, | ||
| "Inputs": [ | ||
| { | ||
| "Name": "Kind", | ||
| "Type": { | ||
| "Kind": "Enum", | ||
| "Values": [ | ||
| "SignatureBinaryClassifierTrainer", | ||
| "SignatureMultiClassClassifierTrainer", | ||
| "SignatureRankerTrainer", | ||
| "SignatureRegressorTrainer", | ||
| "SignatureMultiOutputRegressorTrainer", | ||
| "SignatureAnomalyDetectorTrainer", | ||
| "SignatureClusteringTrainer" | ||
| ] | ||
| }, | ||
| "Desc": "Specifies the trainer kind, which determines the evaluator to be used.", | ||
| "Required": true, | ||
| "SortOrder": 0.0, | ||
| "IsNullable": false, | ||
| "Default": "SignatureBinaryClassifierTrainer" | ||
| }, | ||
| { | ||
| "Name": "OverallMetrics", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Desc": "Overall metrics datasets", | ||
| "Required": false, | ||
| "SortOrder": 1.0, | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "PerInstanceMetrics", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Desc": "Per instance metrics datasets", | ||
| "Required": false, | ||
| "SortOrder": 2.0, | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "ConfusionMatrix", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Desc": "Confusion matrix datasets", | ||
| "Required": false, | ||
| "SortOrder": 3.0, | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "Warnings", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Desc": "Warning datasets", | ||
| "Required": false, | ||
| "SortOrder": 4.0, | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "LabelColumn", | ||
| "Type": "String", | ||
| "Desc": "The label column name", | ||
| "Aliases": [ | ||
| "Label" | ||
| ], | ||
| "Required": false, | ||
| "SortOrder": 4.0, | ||
| "IsNullable": false, | ||
| "Default": "Label" | ||
| } | ||
| ], | ||
| "Outputs": [ | ||
| { | ||
| "Name": "Warnings", | ||
| "Type": "DataView", | ||
| "Desc": "Warning dataset" | ||
| }, | ||
| { | ||
| "Name": "OverallMetrics", | ||
| "Type": "DataView", | ||
| "Desc": "Overall metrics dataset" | ||
| }, | ||
| { | ||
| "Name": "PerInstanceMetrics", | ||
| "Type": "DataView", | ||
| "Desc": "Per instance metrics dataset" | ||
| }, | ||
| { | ||
| "Name": "ConfusionMatrix", | ||
| "Type": "DataView", | ||
| "Desc": "Confusion matrix dataset" | ||
| } | ||
| ] | ||
| }, | ||
| { | ||
| "Name": "Models.CrossValidator", | ||
| "Desc": "Cross validation for general learning", | ||
|
|
@@ -1301,9 +1440,28 @@ | |
| "Name": "Model", | ||
| "Type": "PredictorModel", | ||
| "Desc": "The model", | ||
| "Required": true, | ||
| "Required": false, | ||
| "SortOrder": 1.0, | ||
| "IsNullable": false | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "TransformModel", | ||
| "Type": "TransformModel", | ||
| "Desc": "The transform model", | ||
| "Required": false, | ||
| "SortOrder": 2.0, | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "UseTransformModel", | ||
| "Type": "Bool", | ||
| "Desc": "Indicates to use transform model instead of predictor model.", | ||
| "Required": false, | ||
| "SortOrder": 3.0, | ||
| "IsNullable": false, | ||
| "Default": false | ||
| } | ||
| ] | ||
| }, | ||
|
|
@@ -1367,35 +1525,31 @@ | |
| "Desc": "The final model including the trained predictor model and the model from the transforms, provided as the Input.TransformModel." | ||
| }, | ||
| { | ||
| "Name": "Warnings", | ||
| "Name": "TransformModel", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| "ItemType": "TransformModel" | ||
| }, | ||
| "Desc": "The final model including the trained predictor model and the model from the transforms, provided as the Input.TransformModel." | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is the description correct? #Resolved
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, so we introduced a second option in CV and TrainTest where the final model is of ITransformModel. In reply to: 191033404 [](ancestors = 191033404)
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Update the part of the comment that talks about bundling the predictor model in it.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you please clarify? The comment already talks about predictor being included in the final model. In reply to: 191049219 [](ancestors = 191049219) |
||
| }, | ||
| { | ||
| "Name": "Warnings", | ||
| "Type": "DataView", | ||
| "Desc": "Warning dataset" | ||
| }, | ||
| { | ||
| "Name": "OverallMetrics", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Type": "DataView", | ||
| "Desc": "Overall metrics dataset" | ||
| }, | ||
| { | ||
| "Name": "PerInstanceMetrics", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Type": "DataView", | ||
| "Desc": "Per instance metrics dataset" | ||
| }, | ||
| { | ||
| "Name": "ConfusionMatrix", | ||
| "Type": { | ||
| "Kind": "Array", | ||
| "ItemType": "DataView" | ||
| }, | ||
| "Type": "DataView", | ||
| "Desc": "Confusion matrix dataset" | ||
| } | ||
| ] | ||
|
|
@@ -2904,9 +3058,28 @@ | |
| "Name": "Model", | ||
| "Type": "PredictorModel", | ||
| "Desc": "The model", | ||
| "Required": true, | ||
| "Required": false, | ||
| "SortOrder": 1.0, | ||
| "IsNullable": false | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "TransformModel", | ||
| "Type": "TransformModel", | ||
| "Desc": "Transform model", | ||
| "Required": false, | ||
| "SortOrder": 2.0, | ||
| "IsNullable": false, | ||
| "Default": null | ||
| }, | ||
| { | ||
| "Name": "UseTransformModel", | ||
| "Type": "Bool", | ||
| "Desc": "Indicates to use transform model instead of predictor model.", | ||
| "Required": false, | ||
| "SortOrder": 3.0, | ||
| "IsNullable": false, | ||
| "Default": false | ||
| } | ||
| ] | ||
| }, | ||
|
|
@@ -2960,6 +3133,11 @@ | |
| "Type": "PredictorModel", | ||
| "Desc": "The final model including the trained predictor model and the model from the transforms, provided as the Input.TransformModel." | ||
| }, | ||
| { | ||
| "Name": "TransformModel", | ||
| "Type": "TransformModel", | ||
| "Desc": "The final model including the trained predictor model and the model from the transforms, provided as the Input.TransformModel." | ||
| }, | ||
| { | ||
| "Name": "Warnings", | ||
| "Type": "DataView", | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
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.
So what's the difference between this one and Data.PredictorModelArrayConverter and Data.IDataViewArrayConverter. All of them "Create and array variable" #Resolved
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.
they create array variables of different types, like ITranformModel, IPredictorModel, IDV etc.
In reply to: 191003203 [](ancestors = 191003203)