Allow ComponentCatalog to look for private static Create constuctors

Ivanidzo4ka · web-flow · commit 8ac5ce85cab0 · 2018-09-14T17:42:11.000-07:00
diff --git a/src/Microsoft.ML.Core/ComponentModel/ComponentCatalog.cs b/src/Microsoft.ML.Core/ComponentModel/ComponentCatalog.cs
@@ -336,29 +336,29 @@ private static bool ShouldSkipPath(string path)
             string name = Path.GetFileName(path).ToLowerInvariant();
             switch (name)
             {
-            case "cqo.dll":
-            case "fasttreenative.dll":
-            case "libiomp5md.dll":
-            case "libvw.dll":
-            case "matrixinterf.dll":
-            case "Microsoft.ML.neuralnetworks.gpucuda.dll":
-            case "MklImports.dll":
-            case "microsoft.research.controls.decisiontrees.dll":
-            case "Microsoft.ML.neuralnetworks.sse.dll":
-            case "neuraltreeevaluator.dll":
-            case "optimizationbuilderdotnet.dll":
-            case "parallelcommunicator.dll":
-            case "Microsoft.ML.Runtime.RunTests.dll":
-            case "scopecompiler.dll":
-            case "tbb.dll":
-            case "Internallearnscope.dll":
-            case "unmanagedlib.dll":
-            case "vcclient.dll":
-            case "libxgboost.dll":
-            case "zedgraph.dll":
-            case "__scopecodegen__.dll":
-            case "cosmosClientApi.dll":
-                return true;
+                case "cqo.dll":
+                case "fasttreenative.dll":
+                case "libiomp5md.dll":
+                case "libvw.dll":
+                case "matrixinterf.dll":
+                case "Microsoft.ML.neuralnetworks.gpucuda.dll":
+                case "MklImports.dll":
+                case "microsoft.research.controls.decisiontrees.dll":
+                case "Microsoft.ML.neuralnetworks.sse.dll":
+                case "neuraltreeevaluator.dll":
+                case "optimizationbuilderdotnet.dll":
+                case "parallelcommunicator.dll":
+                case "Microsoft.ML.Runtime.RunTests.dll":
+                case "scopecompiler.dll":
+                case "tbb.dll":
+                case "Internallearnscope.dll":
+                case "unmanagedlib.dll":
+                case "vcclient.dll":
+                case "libxgboost.dll":
+                case "zedgraph.dll":
+                case "__scopecodegen__.dll":
+                case "cosmosClientApi.dll":
+                    return true;
             }
 
             foreach (var s in FilePrefixesToAvoid)
@@ -567,14 +567,14 @@ private static MethodInfo FindInstanceGetter(Type instType, Type loaderType)
 
         private static MethodInfo FindCreateMethod(Type instType, Type loaderType, Type[] parmTypes)
         {
-            var meth = loaderType.GetMethod("Create", parmTypes ?? Type.EmptyTypes);
+            var meth = loaderType.GetMethod("Create", BindingFlags.Public | BindingFlags.Static | BindingFlags.NonPublic | BindingFlags.FlattenHierarchy, null, parmTypes ?? Type.EmptyTypes, null);
             if (meth == null)
                 return null;
             if (meth.DeclaringType != loaderType)
                 return null;
             if (meth.ReturnType != instType)
                 return null;
-            if (!meth.IsPublic || !meth.IsStatic)
+            if (!meth.IsStatic)
                 return null;
             return meth;
         }
diff --git a/src/Microsoft.ML.Data/Transforms/CopyColumnsTransform.cs b/src/Microsoft.ML.Data/Transforms/CopyColumnsTransform.cs
@@ -129,7 +129,8 @@ public sealed class Arguments : TransformInputBase
             public Column[] Column;
         }
 
-        public static IDataView Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        // Factory method for SignatureLoadDataTransform.
+        private static IDataView Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(ctx, nameof(ctx));
@@ -139,7 +140,8 @@ public static IDataView Create(IHostEnvironment env, ModelLoadContext ctx, IData
             return transformer.Transform(input);
         }
 
-        public static CopyColumnsTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        // Factory method for SignatureLoadModel.
+        private static CopyColumnsTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(ctx, nameof(ctx));
@@ -225,7 +227,8 @@ private static VersionInfo GetVersionInfo()
                 loaderSignature: LoaderSignature);
         }
 
-        public static CopyColumnsRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema schema)
+        // Factory method for SignatureLoadRowMapper.
+        private static CopyColumnsRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema schema)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(ctx, nameof(ctx));
diff --git a/src/Microsoft.ML.Data/Transforms/KeyToValueTransform.cs b/src/Microsoft.ML.Data/Transforms/KeyToValueTransform.cs
@@ -112,7 +112,7 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
         /// <summary>
         /// Factory method for SignatureLoadModel.
         /// </summary>
-        public static KeyToValueTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        private static KeyToValueTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(nameof(KeyToValueTransform));
@@ -129,13 +129,13 @@ private KeyToValueTransform(IHost host, ModelLoadContext ctx)
         /// <summary>
         /// Factory method for SignatureLoadDataTransform.
         /// </summary>
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         /// <summary>
         /// Factory method for SignatureLoadRowMapper.
         /// </summary>
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         public override void Save(ModelSaveContext ctx)
diff --git a/src/Microsoft.ML.Data/Transforms/KeyToVectorTransform.cs b/src/Microsoft.ML.Data/Transforms/KeyToVectorTransform.cs
@@ -163,7 +163,7 @@ public override void Save(ModelSaveContext ctx)
         }
 
         // Factory method for SignatureLoadModel.
-        public static KeyToVectorTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        private static KeyToVectorTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(RegistrationName);
@@ -204,7 +204,7 @@ public static IDataTransform Create(IHostEnvironment env, IDataView input, param
              new KeyToVectorTransform(env, columns).MakeDataTransform(input);
 
         // Factory method for SignatureDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, Arguments args, IDataView input)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(args, nameof(args));
@@ -227,11 +227,11 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         protected override IRowMapper MakeRowMapper(ISchema schema) => new Mapper(this, schema);
diff --git a/src/Microsoft.ML.Data/Transforms/Normalizer.cs b/src/Microsoft.ML.Data/Transforms/Normalizer.cs
@@ -405,7 +405,7 @@ public static NormalizerTransformer Create(IHostEnvironment env, ModelLoadContex
         }
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         public override void Save(ModelSaveContext ctx)
diff --git a/src/Microsoft.ML.Data/Transforms/TermTransform.cs b/src/Microsoft.ML.Data/Transforms/TermTransform.cs
@@ -330,7 +330,7 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
         }
 
         // Factory method for SignatureLoadModel.
-        public static TermTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        private static TermTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(RegistrationName);
@@ -385,11 +385,11 @@ private TermTransform(IHost host, ModelLoadContext ctx)
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         /// <summary>
diff --git a/src/Microsoft.ML.ImageAnalytics/ImageGrayscaleTransform.cs b/src/Microsoft.ML.ImageAnalytics/ImageGrayscaleTransform.cs
@@ -98,7 +98,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
                 .MakeDataTransform(input);
         }
 
-        public static ImageGrayscaleTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        // Factory method for SignatureLoadModel.
+        private static ImageGrayscaleTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(RegistrationName);
@@ -113,11 +114,11 @@ private ImageGrayscaleTransform(IHost host, ModelLoadContext ctx)
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         public override void Save(ModelSaveContext ctx)
diff --git a/src/Microsoft.ML.ImageAnalytics/ImageLoaderTransform.cs b/src/Microsoft.ML.ImageAnalytics/ImageLoaderTransform.cs
@@ -84,7 +84,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
                 .MakeDataTransform(data);
         }
 
-        public static ImageLoaderTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        // Factory method for SignatureLoadModel.
+        private static ImageLoaderTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(ctx, nameof(ctx));
@@ -104,11 +105,11 @@ private ImageLoaderTransform(IHost host, ModelLoadContext ctx)
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         protected override void CheckInputColumn(ISchema inputSchema, int col, int srcCol)
diff --git a/src/Microsoft.ML.ImageAnalytics/ImagePixelExtractorTransform.cs b/src/Microsoft.ML.ImageAnalytics/ImagePixelExtractorTransform.cs
@@ -344,7 +344,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
             return new RowToRowMapperTransform(env, input, transformer.MakeRowMapper(input.Schema));
         }
 
-        public static ImagePixelExtractorTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        // Factory method for SignatureLoadModel.
+        private static ImagePixelExtractorTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(RegistrationName);
@@ -369,11 +370,11 @@ private ImagePixelExtractorTransform(IHost host, ModelLoadContext ctx)
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         public override void Save(ModelSaveContext ctx)
diff --git a/src/Microsoft.ML.ImageAnalytics/ImageResizerTransform.cs b/src/Microsoft.ML.ImageAnalytics/ImageResizerTransform.cs
@@ -206,7 +206,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
             return new ImageResizerTransform(env, cols).MakeDataTransform(input);
         }
 
-        public static ImageResizerTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        // Factory method for SignatureLoadModel.
+        private static ImageResizerTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(RegistrationName);
@@ -245,11 +246,11 @@ private ImageResizerTransform(IHost host, ModelLoadContext ctx)
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         public override void Save(ModelSaveContext ctx)
@@ -295,7 +296,7 @@ private sealed class Mapper : MapperBase
             private readonly ImageResizerTransform _parent;
 
             public Mapper(ImageResizerTransform parent, ISchema inputSchema)
-                :base(parent.Host.Register(nameof(Mapper)), parent, inputSchema)
+                : base(parent.Host.Register(nameof(Mapper)), parent, inputSchema)
             {
                 _parent = parent;
             }
diff --git a/src/Microsoft.ML.TensorFlow/TensorflowTransform.cs b/src/Microsoft.ML.TensorFlow/TensorflowTransform.cs
@@ -106,7 +106,7 @@ public static IDataTransform Create(IHostEnvironment env, IDataView input, strin
         }
 
         // Factory method for SignatureLoadModel.
-        public static TensorFlowTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        private static TensorFlowTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(ctx, nameof(ctx));
@@ -142,7 +142,7 @@ public static TensorFlowTransform Create(IHostEnvironment env, ModelLoadContext
         }
 
         // Factory method for SignatureDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, Arguments args, IDataView input)
         {
             Contracts.CheckValue(env, nameof(env));
             env.CheckValue(args, nameof(args));
@@ -153,11 +153,11 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         private TFSession LoadTFSession(byte[] modelBytes)
diff --git a/src/Microsoft.ML.Transforms/KeyToBinaryVectorTransform.cs b/src/Microsoft.ML.Transforms/KeyToBinaryVectorTransform.cs
@@ -107,7 +107,7 @@ public override void Save(ModelSaveContext ctx)
         }
 
         // Factory method for SignatureLoadModel.
-        public static KeyToBinaryVectorTransform Create(IHostEnvironment env, ModelLoadContext ctx)
+        private static KeyToBinaryVectorTransform Create(IHostEnvironment env, ModelLoadContext ctx)
         {
             Contracts.CheckValue(env, nameof(env));
             var host = env.Register(RegistrationName);
@@ -150,11 +150,11 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV
         }
 
         // Factory method for SignatureLoadDataTransform.
-        public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
+        private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)
             => Create(env, ctx).MakeDataTransform(input);
 
         // Factory method for SignatureLoadRowMapper.
-        public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
+        private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)
             => Create(env, ctx).MakeRowMapper(inputSchema);
 
         protected override IRowMapper MakeRowMapper(ISchema schema) => new Mapper(this, schema);

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,8 @@ public sealed class Arguments : TransformInputBase`
`129`	`129`	`public Column[] Column;`
`130`	`130`	`}`
`131`	`131`
`132`		`- public static IDataView Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`132`	`+ // Factory method for SignatureLoadDataTransform.`
	`133`	`+ private static IDataView Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`133`	`134`	`{`
`134`	`135`	`Contracts.CheckValue(env, nameof(env));`
`135`	`136`	`env.CheckValue(ctx, nameof(ctx));`
`@@ -139,7 +140,8 @@ public static IDataView Create(IHostEnvironment env, ModelLoadContext ctx, IData`
`139`	`140`	`return transformer.Transform(input);`
`140`	`141`	`}`
`141`	`142`
`142`		`- public static CopyColumnsTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`143`	`+ // Factory method for SignatureLoadModel.`
	`144`	`+ private static CopyColumnsTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`143`	`145`	`{`
`144`	`146`	`Contracts.CheckValue(env, nameof(env));`
`145`	`147`	`env.CheckValue(ctx, nameof(ctx));`
`@@ -225,7 +227,8 @@ private static VersionInfo GetVersionInfo()`
`225`	`227`	`loaderSignature: LoaderSignature);`
`226`	`228`	`}`
`227`	`229`
`228`		`- public static CopyColumnsRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema schema)`
	`230`	`+ // Factory method for SignatureLoadRowMapper.`
	`231`	`+ private static CopyColumnsRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema schema)`
`229`	`232`	`{`
`230`	`233`	`Contracts.CheckValue(env, nameof(env));`
`231`	`234`	`env.CheckValue(ctx, nameof(ctx));`
Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ public static NormalizerTransformer Create(IHostEnvironment env, ModelLoadContex`
`405`	`405`	`}`
`406`	`406`
`407`	`407`	`// Factory method for SignatureLoadRowMapper.`
`408`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`408`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`409`	`409`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`410`	`410`
`411`	`411`	`public override void Save(ModelSaveContext ctx)`
Original file line number	Diff line number	Diff line change
`@@ -330,7 +330,7 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`330`	`330`	`}`
`331`	`331`
`332`	`332`	`// Factory method for SignatureLoadModel.`
`333`		`- public static TermTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`333`	`+ private static TermTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`334`	`334`	`{`
`335`	`335`	`Contracts.CheckValue(env, nameof(env));`
`336`	`336`	`var host = env.Register(RegistrationName);`
`@@ -385,11 +385,11 @@ private TermTransform(IHost host, ModelLoadContext ctx)`
`385`	`385`	`}`
`386`	`386`
`387`	`387`	`// Factory method for SignatureLoadDataTransform.`
`388`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`388`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`389`	`389`	`=> Create(env, ctx).MakeDataTransform(input);`
`390`	`390`
`391`	`391`	`// Factory method for SignatureLoadRowMapper.`
`392`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`392`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`393`	`393`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`394`	`394`
`395`	`395`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`98`	`98`	`.MakeDataTransform(input);`
`99`	`99`	`}`
`100`	`100`
`101`		`- public static ImageGrayscaleTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`101`	`+ // Factory method for SignatureLoadModel.`
	`102`	`+ private static ImageGrayscaleTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`102`	`103`	`{`
`103`	`104`	`Contracts.CheckValue(env, nameof(env));`
`104`	`105`	`var host = env.Register(RegistrationName);`
`@@ -113,11 +114,11 @@ private ImageGrayscaleTransform(IHost host, ModelLoadContext ctx)`
`113`	`114`	`}`
`114`	`115`
`115`	`116`	`// Factory method for SignatureLoadDataTransform.`
`116`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`117`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`117`	`118`	`=> Create(env, ctx).MakeDataTransform(input);`
`118`	`119`
`119`	`120`	`// Factory method for SignatureLoadRowMapper.`
`120`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`121`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`121`	`122`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`122`	`123`
`123`	`124`	`public override void Save(ModelSaveContext ctx)`
Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`84`	`84`	`.MakeDataTransform(data);`
`85`	`85`	`}`
`86`	`86`
`87`		`- public static ImageLoaderTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`87`	`+ // Factory method for SignatureLoadModel.`
	`88`	`+ private static ImageLoaderTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`88`	`89`	`{`
`89`	`90`	`Contracts.CheckValue(env, nameof(env));`
`90`	`91`	`env.CheckValue(ctx, nameof(ctx));`
`@@ -104,11 +105,11 @@ private ImageLoaderTransform(IHost host, ModelLoadContext ctx)`
`104`	`105`	`}`
`105`	`106`
`106`	`107`	`// Factory method for SignatureLoadDataTransform.`
`107`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`108`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`108`	`109`	`=> Create(env, ctx).MakeDataTransform(input);`
`109`	`110`
`110`	`111`	`// Factory method for SignatureLoadRowMapper.`
`111`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`112`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`112`	`113`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`113`	`114`
`114`	`115`	`protected override void CheckInputColumn(ISchema inputSchema, int col, int srcCol)`
Original file line number	Diff line number	Diff line change
`@@ -344,7 +344,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`344`	`344`	`return new RowToRowMapperTransform(env, input, transformer.MakeRowMapper(input.Schema));`
`345`	`345`	`}`
`346`	`346`
`347`		`- public static ImagePixelExtractorTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`347`	`+ // Factory method for SignatureLoadModel.`
	`348`	`+ private static ImagePixelExtractorTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`348`	`349`	`{`
`349`	`350`	`Contracts.CheckValue(env, nameof(env));`
`350`	`351`	`var host = env.Register(RegistrationName);`
`@@ -369,11 +370,11 @@ private ImagePixelExtractorTransform(IHost host, ModelLoadContext ctx)`
`369`	`370`	`}`
`370`	`371`
`371`	`372`	`// Factory method for SignatureLoadDataTransform.`
`372`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`373`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`373`	`374`	`=> Create(env, ctx).MakeDataTransform(input);`
`374`	`375`
`375`	`376`	`// Factory method for SignatureLoadRowMapper.`
`376`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`377`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`377`	`378`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`378`	`379`
`379`	`380`	`public override void Save(ModelSaveContext ctx)`
Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,8 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`206`	`206`	`return new ImageResizerTransform(env, cols).MakeDataTransform(input);`
`207`	`207`	`}`
`208`	`208`
`209`		`- public static ImageResizerTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`209`	`+ // Factory method for SignatureLoadModel.`
	`210`	`+ private static ImageResizerTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`210`	`211`	`{`
`211`	`212`	`Contracts.CheckValue(env, nameof(env));`
`212`	`213`	`var host = env.Register(RegistrationName);`
`@@ -245,11 +246,11 @@ private ImageResizerTransform(IHost host, ModelLoadContext ctx)`
`245`	`246`	`}`
`246`	`247`
`247`	`248`	`// Factory method for SignatureLoadDataTransform.`
`248`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`249`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`249`	`250`	`=> Create(env, ctx).MakeDataTransform(input);`
`250`	`251`
`251`	`252`	`// Factory method for SignatureLoadRowMapper.`
`252`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`253`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`253`	`254`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`254`	`255`
`255`	`256`	`public override void Save(ModelSaveContext ctx)`
`@@ -295,7 +296,7 @@ private sealed class Mapper : MapperBase`
`295`	`296`	`private readonly ImageResizerTransform _parent;`
`296`	`297`
`297`	`298`	`public Mapper(ImageResizerTransform parent, ISchema inputSchema)`
`298`		`- :base(parent.Host.Register(nameof(Mapper)), parent, inputSchema)`
	`299`	`+ : base(parent.Host.Register(nameof(Mapper)), parent, inputSchema)`
`299`	`300`	`{`
`300`	`301`	`_parent = parent;`
`301`	`302`	`}`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ public static IDataTransform Create(IHostEnvironment env, IDataView input, strin`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`// Factory method for SignatureLoadModel.`
`109`		`- public static TensorFlowTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`109`	`+ private static TensorFlowTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`110`	`110`	`{`
`111`	`111`	`Contracts.CheckValue(env, nameof(env));`
`112`	`112`	`env.CheckValue(ctx, nameof(ctx));`
`@@ -142,7 +142,7 @@ public static TensorFlowTransform Create(IHostEnvironment env, ModelLoadContext`
`142`	`142`	`}`
`143`	`143`
`144`	`144`	`// Factory method for SignatureDataTransform.`
`145`		`- public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataView input)`
	`145`	`+ private static IDataTransform Create(IHostEnvironment env, Arguments args, IDataView input)`
`146`	`146`	`{`
`147`	`147`	`Contracts.CheckValue(env, nameof(env));`
`148`	`148`	`env.CheckValue(args, nameof(args));`
`@@ -153,11 +153,11 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`// Factory method for SignatureLoadDataTransform.`
`156`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`156`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`157`	`157`	`=> Create(env, ctx).MakeDataTransform(input);`
`158`	`158`
`159`	`159`	`// Factory method for SignatureLoadRowMapper.`
`160`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`160`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`161`	`161`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`162`	`162`
`163`	`163`	`private TFSession LoadTFSession(byte[] modelBytes)`
Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ public override void Save(ModelSaveContext ctx)`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`// Factory method for SignatureLoadModel.`
`110`		`- public static KeyToBinaryVectorTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
	`110`	`+ private static KeyToBinaryVectorTransform Create(IHostEnvironment env, ModelLoadContext ctx)`
`111`	`111`	`{`
`112`	`112`	`Contracts.CheckValue(env, nameof(env));`
`113`	`113`	`var host = env.Register(RegistrationName);`
`@@ -150,11 +150,11 @@ public static IDataTransform Create(IHostEnvironment env, Arguments args, IDataV`
`150`	`150`	`}`
`151`	`151`
`152`	`152`	`// Factory method for SignatureLoadDataTransform.`
`153`		`- public static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
	`153`	`+ private static IDataTransform Create(IHostEnvironment env, ModelLoadContext ctx, IDataView input)`
`154`	`154`	`=> Create(env, ctx).MakeDataTransform(input);`
`155`	`155`
`156`	`156`	`// Factory method for SignatureLoadRowMapper.`
`157`		`- public static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
	`157`	`+ private static IRowMapper Create(IHostEnvironment env, ModelLoadContext ctx, ISchema inputSchema)`
`158`	`158`	`=> Create(env, ctx).MakeRowMapper(inputSchema);`
`159`	`159`
`160`	`160`	`protected override IRowMapper MakeRowMapper(ISchema schema) => new Mapper(this, schema);`