PR feedback.

Author: codemzs

src/Microsoft.ML.Data/Model/ModelOperationsCatalog.cs

@@ -204,7 +204,7 @@ public ITransformer Load(string modelPath, out DataViewSchema inputSchema)
         /// <param name="stream">A readable, seekable stream to load from.</param>
         /// <returns>A model of type <see cref="CompositeDataLoader{IMultiStreamSource, ITransformer}"/> containing the loader
         /// and the transformer chain.</returns>
-        public IDataLoader<IMultiStreamSource> Load(Stream stream)
+        public IDataLoader<IMultiStreamSource> LoadDataLoader(Stream stream)
         {
             _env.CheckValue(stream, nameof(stream));
 
@@ -228,10 +228,10 @@ public IDataLoader<IMultiStreamSource> Load(Stream stream)
         /// <param name="modelPath">Path to model.</param>
         /// <returns>A model of type <see cref="CompositeDataLoader{IMultiStreamSource, ITransformer}"/> containing the loader
         /// and the transformer chain.</returns>
-        public IDataLoader<IMultiStreamSource> Load(string modelPath)
+        public IDataLoader<IMultiStreamSource> LoadDataLoader(string modelPath)
         {
             using (var stream = File.OpenRead(modelPath))
-                return Load(stream);
+                return LoadDataLoader(stream);
         }
 
         /// <summary>
@@ -244,7 +244,7 @@ public ITransformer LoadWithDataLoader(Stream stream, out IDataLoader<IMultiStre
         {
             _env.CheckValue(stream, nameof(stream));
 
-            loader = Load(stream);
+            loader = LoadDataLoader(stream);
             if (loader is CompositeDataLoader<IMultiStreamSource, ITransformer> composite)
             {
                 loader = composite.Loader;

test/Microsoft.ML.Functional.Tests/ModelLoading.cs

@@ -74,12 +74,12 @@ public void LoadModelAndExtractPredictor()
             // a transformer model + an input schema, or a transformer model + a data loader.
             var t = _ml.Model.LoadWithDataLoader(compositeLoaderModelPath, out IDataLoader<IMultiStreamSource> l);
             var t1 = _ml.Model.Load(compositeLoaderModelPath, out var s);
-            loadedCompositeLoader = _ml.Model.Load(compositeLoaderModelPath);
+            loadedCompositeLoader = _ml.Model.LoadDataLoader(compositeLoaderModelPath);
 
             // This model can be loaded either as a composite data loader,
             // a transformer model + an input schema, or a transformer model + a data loader.
             var tt = _ml.Model.Load(loaderAndTransformerModelPath, out var ss);
-            var c = _ml.Model.Load(loaderAndTransformerModelPath);
+            var c = _ml.Model.LoadDataLoader(loaderAndTransformerModelPath);
             loadedTransformerModel1 = _ml.Model.LoadWithDataLoader(loaderAndTransformerModelPath, out IDataLoader<IMultiStreamSource> ll);
 
             var gam = ((loadedTransformerModel as ISingleFeaturePredictionTransformer<object>).Model
@@ -120,7 +120,7 @@ public void SaveAndLoadModelWithLoader()
             IDataLoader<IMultiStreamSource> loadedModel;
             ITransformer loadedModelWithoutLoader;
             DataViewSchema loadedSchema;
-            loadedModel = _ml.Model.Load(modelPath);
+            loadedModel = _ml.Model.LoadDataLoader(modelPath);
             loadedModelWithoutLoader = _ml.Model.Load(modelPath, out loadedSchema);
 
             // Without deserializing the loader from the model we lose the slot names.
@@ -292,7 +292,7 @@ private void Load(string filename, out ITransformer loadedWithSchema, out DataVi
 
             try
             {
-                loadedLoader = _ml.Model.Load(filename);
+                loadedLoader = _ml.Model.LoadDataLoader(filename);
             }
             catch (Exception)
             {