Addressed reviewers' comments.

Author: zeahmed

src/Microsoft.ML.Transforms/Text/WrappedTextTransformers.cs

@@ -262,7 +262,7 @@ private static TransformWrapper MakeTransformer(IHostEnvironment env,
     }
 
     /// <summary>
-    /// Produces a bag of counts of ngrams (sequences of consecutive words ) in a given text.
+    /// Produces a bag of counts of ngrams (sequences of consecutive words) in a given text.
     /// It does so by building a dictionary of ngrams and using the id in the dictionary as the index in the bag.
     /// </summary>
     public sealed class WordBagEstimator : TrainedWrapperEstimatorBase
@@ -275,7 +275,7 @@ public sealed class WordBagEstimator : TrainedWrapperEstimatorBase
         private readonly NgramTransform.WeightingCriteria _weighting;
 
         /// <summary>
-        /// Produces a bag of counts of ngrams (sequences of consecutive words ) in <paramref name="inputColumn"/>
+        /// Produces a bag of counts of ngrams (sequences of consecutive words) in <paramref name="inputColumn"/>
         /// and outputs bag of word vector as <paramref name="outputColumn"/>
         /// </summary>
         /// <param name="env">The environment.</param>
@@ -299,7 +299,7 @@ public WordBagEstimator(IHostEnvironment env,
         }
 
         /// <summary>
-        /// Produces a bag of counts of ngrams (sequences of consecutive words ) in <paramref name="inputColumns"/>
+        /// Produces a bag of counts of ngrams (sequences of consecutive words) in <paramref name="inputColumns"/>
         /// and outputs bag of word vector as <paramref name="outputColumn"/>
         /// </summary>
         /// <param name="env">The environment.</param>
@@ -323,7 +323,7 @@ public WordBagEstimator(IHostEnvironment env,
         }
 
         /// <summary>
-        /// Produces a bag of counts of ngrams (sequences of consecutive words ) in <paramref name="columns.inputs"/>
+        /// Produces a bag of counts of ngrams (sequences of consecutive words) in <paramref name="columns.inputs"/>
         /// and outputs bag of word vector for each output in <paramref name="columns.output"/>
         /// </summary>
         /// <param name="env">The environment.</param>
@@ -517,7 +517,7 @@ public sealed class NgramEstimator : TrainedWrapperEstimatorBase
         private readonly NgramTransform.WeightingCriteria _weighting;
 
         /// <summary>
-        /// Produces a bag of counts of ngrams (sequences of consecutive words ) in <paramref name="inputColumn"/>
+        /// Produces a bag of counts of ngrams (sequences of consecutive words) in <paramref name="inputColumn"/>
         /// and outputs bag of word vector as <paramref name="outputColumn"/>
         /// </summary>
         /// <param name="env">The environment.</param>
@@ -541,7 +541,7 @@ public NgramEstimator(IHostEnvironment env,
         }
 
         /// <summary>
-        /// Produces a bag of counts of ngrams (sequences of consecutive words ) in <paramref name="columns.inputs"/>
+        /// Produces a bag of counts of ngrams (sequences of consecutive words) in <paramref name="columns.inputs"/>
         /// and outputs bag of word vector for each output in <paramref name="columns.output"/>
         /// </summary>
         /// <param name="env">The environment.</param>

test/Microsoft.ML.Tests/Transformers/TextFeaturizerTests.cs

@@ -140,7 +140,10 @@ public void WordBagWorkout()
 
             var est = new WordBagEstimator(Env, "text", "bag_of_words").
                 Append(new WordHashBagEstimator(Env, "text", "bag_of_wordshash"));
-            //TestEstimatorCore(est, data.AsDynamic, invalidInput: invalidData.AsDynamic);
+            
+            // The following call fails because of the following issue
+            // https://github.com/dotnet/machinelearning/issues/969
+            // TestEstimatorCore(est, data.AsDynamic, invalidInput: invalidData.AsDynamic);
 
             var outputPath = GetOutputPath("Text", "bag_of_words.tsv");
             using (var ch = Env.Start("save"))
@@ -175,7 +178,10 @@ public void NgramWorkout()
                 .Append(new TermEstimator(Env, "text", "terms"))
                 .Append(new NgramEstimator(Env, "terms", "ngrams"))
                 .Append(new NgramHashEstimator(Env, "terms", "ngramshash"));
-            //TestEstimatorCore(est, data.AsDynamic, invalidInput: invalidData.AsDynamic);
+            
+            // The following call fails because of the following issue
+            // https://github.com/dotnet/machinelearning/issues/969
+            // TestEstimatorCore(est, data.AsDynamic, invalidInput: invalidData.AsDynamic);
 
             var outputPath = GetOutputPath("Text", "ngrams.tsv");
             using (var ch = Env.Start("save"))