Ivan comments

Author: Tom Finley

src/Microsoft.ML.Data/Commands/TrainCommand.cs

@@ -522,7 +522,6 @@ public static bool AddNormalizerIfNeeded(IHostEnvironment env, IChannel ch, ITra
                     }
                 }
                 ch.Info("Automatically adding a MinMax normalization transform, use 'norm=Warn' or 'norm=No' to turn this behavior off.");
-                // REVIEW: This verbose constructor should be replaced with zeahmed's enhancements once #405 is committed.
                 IDataView ApplyNormalizer(IHostEnvironment innerEnv, IDataView input)
                     => NormalizeTransform.Create(innerEnv, new NormalizeTransform.MinMaxArguments()
                     {

src/Microsoft.ML.FastTree/FastTree.cs

@@ -1347,18 +1347,16 @@ private Dataset Construct(RoleMappedData examples, ref int numExamples, int maxB
                         data = new LabelConvertTransform(Host, convArgs, data);
                     }
                     // Convert the group column, if one exists.
-                    var groupInfo = examples.Schema.Group;
-                    if (groupInfo != null)
+                    if (examples.Schema.Group != null)
                     {
                         var convArgs = new ConvertTransform.Arguments();
                         var convCol = new ConvertTransform.Column
                         {
                             ResultType = DataKind.U8
                         };
-                        convCol.Name = convCol.Source = groupInfo.Name;
+                        convCol.Name = convCol.Source = examples.Schema.Group.Name;
                         convArgs.Column = new ConvertTransform.Column[] { convCol };
                         data = new ConvertTransform(Host, convArgs, data);
-                        groupInfo = ColumnInfo.CreateFromName(data.Schema, convCol.Name, "converted group id");
                     }
 
                     // Since we've passed it through a few transforms, reconstitute the mapping on the