Add support for reflecting internal fields from argument types

Author: mareklinka

src/Microsoft.ML.Core/EntryPoints/EntryPointUtils.cs

@@ -4,6 +4,7 @@
 
 using System;
 using System.Linq;
+using System.Reflection;
 using Microsoft.ML.CommandLine;
 using Microsoft.ML.Data;
 using Microsoft.ML.Internal.Utilities;
@@ -42,15 +43,15 @@ public static bool IsValueWithinRange(this TlcModule.RangeAttribute range, objec
         }
 
         /// <summary>
-        /// Performs checks on an EntryPoint input class equivilent to the checks that are done
+        /// Performs checks on an EntryPoint input class equivalent to the checks that are done
         /// when parsing a JSON EntryPoint graph.
         ///
         /// Call this method from EntryPoint methods to ensure that range and required checks are performed
         /// in a consistent manner when EntryPoints are created directly from code.
         /// </summary>
         public static void CheckInputArgs(IExceptionContext ectx, object args)
         {
-            foreach (var fieldInfo in args.GetType().GetFields())
+            foreach (var fieldInfo in args.GetType().GetFields(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance))
             {
                 var attr = fieldInfo.GetCustomAttributes(typeof(ArgumentAttribute), false).FirstOrDefault()
                     as ArgumentAttribute;

src/Microsoft.ML.EntryPoints/JsonUtils/JsonManifestUtils.cs

@@ -154,7 +154,7 @@ private static JArray BuildInputManifest(IExceptionContext ectx, Type inputType,
             var defaults = Activator.CreateInstance(inputType);
 
             var inputs = new List<KeyValuePair<Double, JObject>>();
-            foreach (var fieldInfo in inputType.GetFields())
+            foreach (var fieldInfo in inputType.GetFields(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance))
             {
                 var inputAttr = fieldInfo.GetCustomAttributes(typeof(ArgumentAttribute), false).FirstOrDefault() as ArgumentAttribute;
                 if (inputAttr == null || inputAttr.Visibility == ArgumentAttribute.VisibilityType.CmdLineOnly)
@@ -276,7 +276,7 @@ private static JArray BuildOutputManifest(IExceptionContext ectx, Type outputTyp
             if (outputType.IsGenericType && outputType.GetGenericTypeDefinition() == typeof(CommonOutputs.MacroOutput<>))
                 outputType = outputType.GetGenericArguments()[0];
 
-            foreach (var fieldInfo in outputType.GetFields())
+            foreach (var fieldInfo in outputType.GetFields(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance))
             {
                 var outputAttr = fieldInfo.GetCustomAttributes(typeof(TlcModule.OutputAttribute), false)
                     .FirstOrDefault() as TlcModule.OutputAttribute;
@@ -476,7 +476,7 @@ private static JToken BuildComponentToken(IExceptionContext ectx, IComponentFact
             // Iterate over all fields of the factory object, and compare the values with the defaults.
             // If the value differs, insert it into the settings object.
             bool anyValue = false;
-            foreach (var fieldInfo in type.GetFields())
+            foreach (var fieldInfo in type.GetFields(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance))
             {
                 var attr = fieldInfo.GetCustomAttributes(typeof(ArgumentAttribute), false).FirstOrDefault()
                     as ArgumentAttribute;