address comments and entry points test

Author: Ivan Matantsev

docs/samples/Microsoft.ML.Samples/Dynamic/Trainers/BinaryClassification/AveragedPerceptronWithOptions.cs

@@ -25,7 +25,7 @@ public static void Example()
             {
                 LossFunction = new SmoothedHingeLoss.Options(),
                 LearningRate = 0.1f,
-                LazyUpdates = false,
+                LazyUpdate = false,
                 RecencyGain = 0.1f,
                 NumberOfIterations = 10
             };

src/Microsoft.ML.StandardLearners/Standard/Online/AveragedLinear.cs

@@ -57,8 +57,8 @@ public abstract class AveragedLinearOptions : OnlineLinearOptions
         /// <see langword="false" /> to update averaged weights on every example.
         /// Default is <see langword="true" />.
         /// </value>
-        [Argument(ArgumentType.AtMostOnce, HelpText = "Instead of updating averaged weights on every example, only update when loss is nonzero", ShortName = "lazy")]
-        public bool LazyUpdates = true;
+        [Argument(ArgumentType.AtMostOnce, HelpText = "Instead of updating averaged weights on every example, only update when loss is nonzero", ShortName = "lazy,DoLazyUpdates")]
+        public bool LazyUpdate = true;
 
         /// <summary>
         /// The L2 weight for <a href='tmpurl_regularization'>regularization</a>.
@@ -85,7 +85,7 @@ public abstract class AveragedLinearOptions : OnlineLinearOptions
         /// <see langword="false" /> means <see cref="RecencyGain"/> is additive.
         /// Default is <see langword="false" />.
         /// </value>
-        [Argument(ArgumentType.AtMostOnce, HelpText = "Whether Recency Gain is multiplicative (vs. additive)", ShortName = "rgm")]
+        [Argument(ArgumentType.AtMostOnce, HelpText = "Whether Recency Gain is multiplicative (vs. additive)", ShortName = "rgm,RecencyGainMulti")]
         public bool RecencyGainMultiplicative = false;
 
         /// <summary>
@@ -186,7 +186,7 @@ public override void FinishIteration(IChannel ch)
                 // Finalize things
                 if (Averaged)
                 {
-                    if (_args.LazyUpdates && NumNoUpdates > 0)
+                    if (_args.LazyUpdate && NumNoUpdates > 0)
                     {
                         // Update the total weights to include the final loss=0 updates
                         VectorUtils.AddMult(in Weights, NumNoUpdates * WeightsScale, ref TotalWeights);
@@ -224,7 +224,7 @@ public override void ProcessDataInstance(IChannel ch, in VBuffer<float> feat, fl
                 if (loss != 0 || _args.L2Regularization > 0)
                 {
                     // If doing lazy weights, we need to update the totalWeights and totalBias before updating weights/bias
-                    if (_args.LazyUpdates && _args.Averaged && NumNoUpdates > 0 && TotalMultipliers * _args.AveragedTolerance <= PendingMultipliers)
+                    if (_args.LazyUpdate && _args.Averaged && NumNoUpdates > 0 && TotalMultipliers * _args.AveragedTolerance <= PendingMultipliers)
                     {
                         VectorUtils.AddMult(in Weights, NumNoUpdates * WeightsScale, ref TotalWeights);
                         TotalBias += Bias * NumNoUpdates * WeightsScale;
@@ -251,7 +251,7 @@ public override void ProcessDataInstance(IChannel ch, in VBuffer<float> feat, fl
                 // Add to averaged weights and increment the count.
                 if (Averaged)
                 {
-                    if (!_args.LazyUpdates)
+                    if (!_args.LazyUpdate)
                         IncrementAverageNonLazy();
                     else
                         NumNoUpdates++;
@@ -307,7 +307,7 @@ private protected AveragedLinearTrainer(AveragedLinearOptions options, IHostEnvi
             Contracts.CheckUserArg(options.RecencyGain >= 0, nameof(options.RecencyGain), UserErrorNonNegative);
             Contracts.CheckUserArg(options.AveragedTolerance >= 0, nameof(options.AveragedTolerance), UserErrorNonNegative);
             // Verify user didn't specify parameters that conflict
-            Contracts.Check(!options.LazyUpdates || !options.RecencyGainMultiplicative && options.RecencyGain == 0, "Cannot have both recency gain and lazy updates.");
+            Contracts.Check(!options.LazyUpdate || !options.RecencyGainMultiplicative && options.RecencyGain == 0, "Cannot have both recency gain and lazy updates.");
 
             AveragedLinearTrainerOptions = options;
         }

src/Microsoft.ML.StandardLearners/Standard/Online/LinearSvm.cs

@@ -69,7 +69,7 @@ public sealed class Options : OnlineLinearOptions
             /// <summary>
             /// Column to use for example weight.
             /// </summary>
-            [Argument(ArgumentType.AtMostOnce, HelpText = "Column to use for example weight", ShortName = "weight", SortOrder = 4, Visibility = ArgumentAttribute.VisibilityType.EntryPointsOnly)]
+            [Argument(ArgumentType.AtMostOnce, HelpText = "Column to use for example weight", ShortName = "weight,WeightColumn", SortOrder = 4, Visibility = ArgumentAttribute.VisibilityType.EntryPointsOnly)]
             public string ExampleWeightColumnName = null;
         }
 

test/BaselineOutput/Common/EntryPoints/core_manifest.json

@@ -4286,11 +4286,12 @@
           }
         },
         {
-          "Name": "L2RegularizerWeight",
+          "Name": "L2Regularization",
           "Type": "Float",
           "Desc": "L2 Regularization Weight",
           "Aliases": [
-            "reg"
+            "reg",
+            "L2RegularizerWeight"
           ],
           "Required": false,
           "SortOrder": 50.0,
@@ -4377,11 +4378,12 @@
           "Default": null
         },
         {
-          "Name": "DoLazyUpdates",
+          "Name": "LazyUpdate",
           "Type": "Bool",
           "Desc": "Instead of updating averaged weights on every example, only update when loss is nonzero",
           "Aliases": [
-            "lazy"
+            "lazy",
+            "DoLazyUpdates"
           ],
           "Required": false,
           "SortOrder": 150.0,
@@ -4401,11 +4403,12 @@
           "Default": 0.0
         },
         {
-          "Name": "RecencyGainMulti",
+          "Name": "RecencyGainMultiplicative",
           "Type": "Bool",
           "Desc": "Whether Recency Gain is multiplicative (vs. additive)",
           "Aliases": [
-            "rgm"
+            "rgm",
+            "RecencyGainMulti"
           ],
           "Required": false,
           "SortOrder": 150.0,
@@ -13113,11 +13116,12 @@
           "Default": "Label"
         },
         {
-          "Name": "WeightColumn",
+          "Name": "ExampleWeightColumnName",
           "Type": "String",
           "Desc": "Column to use for example weight",
           "Aliases": [
-            "weight"
+            "weight",
+            "WeightColumn"
           ],
           "Required": false,
           "SortOrder": 4.0,
@@ -14214,11 +14218,12 @@
           }
         },
         {
-          "Name": "L2RegularizerWeight",
+          "Name": "L2Regularization",
           "Type": "Float",
           "Desc": "L2 Regularization Weight",
           "Aliases": [
-            "reg"
+            "reg",
+            "L2RegularizerWeight"
           ],
           "Required": false,
           "SortOrder": 50.0,
@@ -14282,11 +14287,12 @@
           "Default": null
         },
         {
-          "Name": "DoLazyUpdates",
+          "Name": "LazyUpdate",
           "Type": "Bool",
           "Desc": "Instead of updating averaged weights on every example, only update when loss is nonzero",
           "Aliases": [
-            "lazy"
+            "lazy",
+            "DoLazyUpdates"
           ],
           "Required": false,
           "SortOrder": 150.0,
@@ -14306,11 +14312,12 @@
           "Default": 0.0
         },
         {
-          "Name": "RecencyGainMulti",
+          "Name": "RecencyGainMultiplicative",
           "Type": "Bool",
           "Desc": "Whether Recency Gain is multiplicative (vs. additive)",
           "Aliases": [
-            "rgm"
+            "rgm",
+            "RecencyGainMulti"
           ],
           "Required": false,
           "SortOrder": 150.0,