default initialization for ignore columns (dotnet#302)

* default initialization * adde null check
Dmitry-A · Mar 20, 2019 · d2c50a7 · d2c50a7
1 parent 30046a1
commit d2c50a7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/mlnet/Commands/CommandDefinitions.cs b/src/mlnet/Commands/CommandDefinitions.cs
@@ -51,7 +51,7 @@ internal static System.CommandLine.Command New(ICommandHandler handler)
                 {
                     return "The following options are mutually exclusive please provide only one : --label-column-name, --label-column-index";
                 }
-                if (sym.Children.Contains("--label-column-index") && sym.Children["--ignore-columns"].Arguments.Count > 0)
+                if (sym.Children.Contains("--label-column-index") && sym.Children["--ignore-columns"]?.Arguments.Count > 0)
                 {
                     return "Currently we don't support specifying --ignore-columns in conjunction with --label-column-index";
                 }

diff --git a/src/mlnet/Commands/New/NewCommandSettings.cs b/src/mlnet/Commands/New/NewCommandSettings.cs
@@ -33,7 +33,7 @@ public class NewCommandSettings
 
         public string Cache { get; set; }
 
-        public List<string> IgnoreColumns { get; set; }
+        public List<string> IgnoreColumns { get; set; } = new List<string>();
 
     }
 }