From a8a9240a904dbbbd2baaaf99bfde4df76f689098 Mon Sep 17 00:00:00 2001 From: Srujan Saggam <41802116+srsaggam@users.noreply.github.com> Date: Tue, 19 Mar 2019 17:18:06 -0700 Subject: [PATCH] default initialization for ignore columns (#302) * default initialization * adde null check --- src/mlnet/Commands/CommandDefinitions.cs | 2 +- src/mlnet/Commands/New/NewCommandSettings.cs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mlnet/Commands/CommandDefinitions.cs b/src/mlnet/Commands/CommandDefinitions.cs index 972fe2f388d..535f878b20d 100644 --- 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 index 5ad3ef96321..686e427aebf 100644 --- 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 IgnoreColumns { get; set; } + public List IgnoreColumns { get; set; } = new List(); } }