Changed type to useImageType in LoadImages(). Changed appropriate variable names in ImageClassificationTransform.cs

Author: harshithapv

src/Microsoft.ML.Dnn/ImageClassificationTransform.cs

@@ -813,8 +813,8 @@ public Mapper(ImageClassificationTransformer parent, DataViewSchema inputSchema)
             private class OutputCache
             {
                 public long Position;
-                private ValueGetter<VBuffer<byte>> _imageBufGetter;
-                private VBuffer<byte> _imageBuf;
+                private ValueGetter<VBuffer<byte>> _imageGetter;
+                private VBuffer<byte> _image;
                 private Runner _runner;
                 private ImageProcessor _imageProcessor;
                 public UInt32 PredictedLabel { get; set; }
@@ -823,8 +823,8 @@ private class OutputCache
 
                 public OutputCache(DataViewRow input, ImageClassificationTransformer transformer)
                 {
-                    _imageBuf = default;
-                    _imageBufGetter = input.GetGetter<VBuffer<byte>>(input.Schema[transformer._inputs[0]]);
+                    _image = default;
+                    _imageGetter = input.GetGetter<VBuffer<byte>>(input.Schema[transformer._inputs[0]]);
                     _runner = new Runner(transformer._session);
                     _runner.AddInput(transformer._inputTensorName);
                     _runner.AddOutputs(transformer._softmaxTensorName);
@@ -841,8 +841,8 @@ public void UpdateCacheIfNeeded()
                         if (_inputRow.Position != Position)
                         {
                             Position = _inputRow.Position;
-                            _imageBufGetter(ref _imageBuf);
-                            var processedTensor = _imageProcessor.ProcessImage(_imageBuf);
+                            _imageGetter(ref _image);
+                            var processedTensor = _imageProcessor.ProcessImage(_image);
                             var outputTensor = _runner.AddInput(processedTensor, 0).Run();
                             ClassProbabilities = outputTensor[0].ToArray<float>();
                             PredictedLabel = (UInt32)outputTensor[1].ToArray<long>()[0];

src/Microsoft.ML.ImageAnalytics/ExtensionsCatalog.cs

@@ -107,15 +107,15 @@ internal static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog,
         /// <param name="inputColumnName">Name of the column with paths to the images to load.
         /// This estimator operates over text data.</param>
         /// <param name="imageFolder">Folder where to look for images.</param>
-        /// <param name="type">Image type flag - If true loads image as a VectorDataView type else loads image as ImageDataViewType. Defaults to ImageDataViewType if not specified or false.</param>
+        /// <param name="useImageType">Image type flag - If true loads image as a VectorDataView type else loads image as ImageDataViewType. Defaults to ImageDataViewType if not specified or false.</param>
         /// <example>
         /// <format type="text/markdown">
         /// <![CDATA[
         ///  [!code-csharp[LoadImages](~/../docs/samples/docs/samples/Microsoft.ML.Samples/Dynamic/Transforms/ImageAnalytics/LoadImages.cs)]
         /// ]]></format>
         /// </example>
-        public static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog, string outputColumnName, string imageFolder, bool type, string inputColumnName = null)
-           => new ImageLoadingEstimator(CatalogUtils.GetEnvironment(catalog), imageFolder, type, new[] { (outputColumnName, inputColumnName ?? outputColumnName) });
+        public static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog, string outputColumnName, string imageFolder, bool useImageType, string inputColumnName = null)
+           => new ImageLoadingEstimator(CatalogUtils.GetEnvironment(catalog), imageFolder, useImageType, new[] { (outputColumnName, inputColumnName ?? outputColumnName) });
 
         /// <summary>
         /// Loads the images from the <see cref="ImageLoadingTransformer.ImageFolder" /> into memory.
@@ -129,7 +129,7 @@ public static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog, s
         /// </remarks>
         /// <param name="catalog">The transform's catalog.</param>
         /// <param name="imageFolder">Folder where to look for images.</param>
-        /// <param name="type">Image type flag - If true loads image as a VectorDataView type else loads image as ImageDataViewType. Defaults to ImageDataViewType if not specified or false.</param>
+        /// <param name="useImageType ">Image type flag - If true loads image as a VectorDataView type else loads image as ImageDataViewType. Defaults to ImageDataViewType if not specified or false.</param>
         /// <param name="columns">Specifies the names of the input columns for the transformation, and their respective output column names.</param>
         /// <example>
         /// <format type="text/markdown">
@@ -138,11 +138,11 @@ public static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog, s
         /// ]]></format>
         /// </example>
         [BestFriend]
-        internal static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog, string imageFolder, bool type, params InputOutputColumnPair[] columns)
+        internal static ImageLoadingEstimator LoadImages(this TransformsCatalog catalog, string imageFolder, bool useImageType, params InputOutputColumnPair[] columns)
         {
             var env = CatalogUtils.GetEnvironment(catalog);
             env.CheckValue(columns, nameof(columns));
-            return new ImageLoadingEstimator(env, imageFolder, type, InputOutputColumnPair.ConvertToValueTuples(columns));
+            return new ImageLoadingEstimator(env, imageFolder, useImageType, InputOutputColumnPair.ConvertToValueTuples(columns));
         }
 
         /// <summary>