Review comments.

Author: TomFinley

docs/samples/Microsoft.ML.Samples/Dynamic/Calibrator.cs

@@ -1,8 +1,4 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-// See the LICENSE file in the project root for more information.
-
-using System;
+using System;
 using System.Linq;
 using Microsoft.ML.Calibrator;
 using Microsoft.ML.Data;

src/Microsoft.ML.Core/BestFriendAttribute.cs

@@ -13,7 +13,7 @@ namespace Microsoft.ML
 #endif
 {
     /// <summary>
-    /// Intended to be applied to types and members marked as internal to indicate that friend access of this
+    /// Intended to be applied to types and members with internal scope to indicate that friend access of this
     /// internal item is OK from another assembly. This restriction applies only to assemblies that declare the
     /// <see cref="WantsToBeBestFriendsAttribute"/> assembly level attribute. Note that this attribute is not
     /// transferrable: an internal member with this attribute does not somehow make a containing internal type

src/Microsoft.ML.Data/Prediction/Calibrator.cs

@@ -1703,6 +1703,7 @@ public override ICalibrator CreateCalibrator(IChannel ch)
     }
 
     /// <summary>
+    /// The pair-adjacent violators calibrator.
     /// The function that is implemented by this calibrator is:
     /// f(x) = v_i, if minX_i &lt;= x &lt;= maxX_i
     ///      = linear interpolate between v_i and v_i+1, if maxX_i &lt; x &lt; minX_i+1

src/Microsoft.ML.Data/Prediction/CalibratorCatalog.cs

@@ -260,7 +260,7 @@ protected override Delegate MakeGetter(Row input, int iinfo, Func<int, bool> act
     }
 
     /// <summary>
-    /// The PlattCalibratorEstimator.
+    /// The Platt calibrator estimator.
     /// </summary>
     /// <remarks>
     /// For the usage pattern see the example in <see cref="CalibratorEstimatorBase{TICalibrator}"/>.
@@ -291,6 +291,8 @@ private protected override CalibratorTransformer<PlattCalibrator> Create(IHostEn
 
     /// <summary>
     /// Obtains the probability values by applying the sigmoid:  f(x) = 1 / (1 + exp(-slope * x + offset).
+    /// Note that unlike, say, <see cref="PlattCalibratorEstimator"/>, the fit function here is trivial
+    /// and just "fits" a calibrator with the provided parameters.
     /// </summary>
     /// <remarks>
     /// For the usage pattern see the example in <see cref="CalibratorEstimatorBase{TICalibrator}"/>.
@@ -347,7 +349,7 @@ internal PlattCalibratorTransformer(IHostEnvironment env, ModelLoadContext ctx)
     }
 
     /// <summary>
-    /// The naive binning-based calibratorEstimator.
+    /// The naive binning-based calbirator estimator.
     /// </summary>
     /// <remarks>
     /// It divides the range of the outputs into equally sized bins. In each bin,
@@ -399,7 +401,7 @@ internal NaiveCalibratorTransformer(IHostEnvironment env, ModelLoadContext ctx)
     }
 
     /// <summary>
-    /// The PavCalibratorEstimator.
+    /// The pair-adjacent violators calibrator estimator.
     /// </summary>
     /// <remarks>
     /// For the usage pattern see the example in <see cref="CalibratorEstimatorBase{TICalibrator}"/>.