Some spaces and extra lines + bug in output path (dotnet#204)

srsaggam · Dmitry-A · commit 6fa307a93b1b · 2019-08-22T14:40:10.000-07:00
* nit picks

* nit picks

* fix test
diff --git a/src/mlnet.Test/ApprovalTests/ConsoleCodeGeneratorTests.GeneratedTrainCodeTest.approved.txt b/src/mlnet.Test/ApprovalTests/ConsoleCodeGeneratorTests.GeneratedTrainCodeTest.approved.txt
@@ -83,7 +83,7 @@ namespace MyNamespace
             return trainedModel;
         }
 
-        // (OPTIONAL) Try/test a single prediction by loding the model from the file, first.
+        // (OPTIONAL) Try/test a single prediction by loading the model from the file, first.
         private static void TestSinglePrediction(MLContext mlContext)
         {
             //Load data to test. Could be any test data. For demonstration purpose train data is used here.
diff --git a/src/mlnet/Commands/CommandDefinitions.cs b/src/mlnet/Commands/CommandDefinitions.cs
@@ -93,7 +93,7 @@ Option Name() =>
 
             Option OutputPath() =>
               new Option(new List<string>() { "--output-path" }, "Output folder path.",
-             new Argument<DirectoryInfo>(defaultValue: new DirectoryInfo(".\\Sample")));
+             new Argument<DirectoryInfo>(defaultValue: new DirectoryInfo(".")));
 
             Option HasHeader() =>
              new Option(new List<string>() { "--has-header" }, "Specifies if the dataset has header or not.",
diff --git a/src/mlnet/Templates/Console/MLCodeGen.cs b/src/mlnet/Templates/Console/MLCodeGen.cs
@@ -109,9 +109,6 @@ private static ITransformer BuildTrainEvaluateAndSaveModel(MLContext mlContext)
                                              { Write("\r\n                                      .Append(");
                                              }
                                              Write("mlContext.Transforms."+Transforms[i]);
-                                             if(i>0)
-                                             { Write(")\r\n");
-                                             }
                                          }
             this.Write(";\r\n");
 }
@@ -181,7 +178,7 @@ private static ITransformer BuildTrainEvaluateAndSaveModel(MLContext mlContext)
             return trainedModel;
         }
 
-        // (OPTIONAL) Try/test a single prediction by loding the model from the file, first.
+        // (OPTIONAL) Try/test a single prediction by loading the model from the file, first.
         private static void TestSinglePrediction(MLContext mlContext)
         {
             //Load data to test. Could be any test data. For demonstration purpose train data is used here.
diff --git a/src/mlnet/Templates/Console/MLCodeGen.tt b/src/mlnet/Templates/Console/MLCodeGen.tt
@@ -72,9 +72,6 @@ namespace <#= Namespace #>
                                              { Write("\r\n                                      .Append(");
                                              }
                                              Write("mlContext.Transforms."+Transforms[i]);
-                                             if(i>0)
-                                             { Write(")\r\n");
-                                             }
                                          }#>;
 <#}#>
 
@@ -127,7 +124,7 @@ else{#>
             return trainedModel;
         }
 
-        // (OPTIONAL) Try/test a single prediction by loding the model from the file, first.
+        // (OPTIONAL) Try/test a single prediction by loading the model from the file, first.
         private static void TestSinglePrediction(MLContext mlContext)
         {
             //Load data to test. Could be any test data. For demonstration purpose train data is used here.

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ namespace MyNamespace`
`83`	`83`	`return trainedModel;`
`84`	`84`	`}`
`85`	`85`
`86`		`- // (OPTIONAL) Try/test a single prediction by loding the model from the file, first.`
	`86`	`+ // (OPTIONAL) Try/test a single prediction by loading the model from the file, first.`
`87`	`87`	`private static void TestSinglePrediction(MLContext mlContext)`
`88`	`88`	`{`
`89`	`89`	`//Load data to test. Could be any test data. For demonstration purpose train data is used here.`
Original file line number	Diff line number	Diff line change
`@@ -109,9 +109,6 @@ private static ITransformer BuildTrainEvaluateAndSaveModel(MLContext mlContext)`
`109`	`109`	`{ Write("\r\n .Append(");`
`110`	`110`	`}`
`111`	`111`	`Write("mlContext.Transforms."+Transforms[i]);`
`112`		`- if(i>0)`
`113`		`- { Write(")\r\n");`
`114`		`- }`
`115`	`112`	`}`
`116`	`113`	`this.Write(";\r\n");`
`117`	`114`	`}`
`@@ -181,7 +178,7 @@ private static ITransformer BuildTrainEvaluateAndSaveModel(MLContext mlContext)`
`181`	`178`	`return trainedModel;`
`182`	`179`	`}`
`183`	`180`
`184`		`- // (OPTIONAL) Try/test a single prediction by loding the model from the file, first.`
	`181`	`+ // (OPTIONAL) Try/test a single prediction by loading the model from the file, first.`
`185`	`182`	`private static void TestSinglePrediction(MLContext mlContext)`
`186`	`183`	`{`
`187`	`184`	`//Load data to test. Could be any test data. For demonstration purpose train data is used here.`
Original file line number	Diff line number	Diff line change
`@@ -72,9 +72,6 @@ namespace <#= Namespace #>`
`72`	`72`	`{ Write("\r\n .Append(");`
`73`	`73`	`}`
`74`	`74`	`Write("mlContext.Transforms."+Transforms[i]);`
`75`		`- if(i>0)`
`76`		`- { Write(")\r\n");`
`77`		`- }`
`78`	`75`	`}#>;`
`79`	`76`	`<#}#>`
`80`	`77`
`@@ -127,7 +124,7 @@ else{#>`
`127`	`124`	`return trainedModel;`
`128`	`125`	`}`
`129`	`126`
`130`		`- // (OPTIONAL) Try/test a single prediction by loding the model from the file, first.`
	`127`	`+ // (OPTIONAL) Try/test a single prediction by loading the model from the file, first.`
`131`	`128`	`private static void TestSinglePrediction(MLContext mlContext)`
`132`	`129`	`{`
`133`	`130`	`//Load data to test. Could be any test data. For demonstration purpose train data is used here.`