Skip to content

Commit 4dd116c

Browse files
committed
PR feedback.
1 parent 0bef3e5 commit 4dd116c

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/Microsoft.ML.PCA/PCACatalog.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ internal static PrincipalComponentAnalyzer ProjectToPrincipalComponents(this Tra
4040
=> new PrincipalComponentAnalyzer(CatalogUtils.GetEnvironment(catalog), columns);
4141

4242
/// <summary>
43-
/// Creates a <see cref="RandomizedPcaTrainer"/>, which trains an approximate principal component analysis (PCA) model using randomized SVD algorithm.
43+
/// Create <see cref="RandomizedPcaTrainer"/>, which trains an approximate principal component analysis (PCA) model using randomized SVD algorithm.
4444
/// </summary>
4545
/// <param name="catalog">The anomaly detection catalog trainer object.</param>
4646
/// <param name="featureColumnName">The name of the feature column. The column data must be a known-sized vector of <see cref="System.Single"/>.</param>
@@ -69,7 +69,7 @@ public static RandomizedPcaTrainer RandomizedPca(this AnomalyDetectionCatalog.An
6969
}
7070

7171
/// <summary>
72-
/// TCreates a <see cref="RandomizedPcaTrainer"/>, which trains an approximate principal component analysis (PCA) model using randomized SVD algorithm with advanced options.
72+
/// Create <see cref="RandomizedPcaTrainer"/> using advanced options, which trains an approximate principal component analysis (PCA) model using randomized SVD algorithm.
7373
/// </summary>
7474
/// <param name="catalog">The anomaly detection catalog trainer object.</param>
7575
/// <param name="options">Advanced options to the algorithm.</param>

src/Microsoft.ML.PCA/PcaTrainer.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ namespace Microsoft.ML.Trainers
4646
/// | | |
4747
/// | -- | -- |
4848
/// | Machine learning task | Anomaly Detection |
49-
/// | Is normalization required? | Yes |
49+
/// | Is normalization required? | No |
5050
/// | Is caching required? | No |
5151
/// | Required NuGet in addition to Microsoft.ML | None |
5252
///
@@ -55,8 +55,8 @@ namespace Microsoft.ML.Trainers
5555
/// ]]>
5656
/// </format>
5757
/// </remarks>
58-
/// <seealso cref="Microsoft.ML.PcaCatalog.RandomizedPca(Microsoft.ML.AnomalyDetectionCatalog.AnomalyDetectionTrainers,System.String,System.String,System.Int32,System.Int32,System.Boolean,System.Nullable{System.Int32})"/>
59-
/// <seealso cref="Microsoft.ML.PcaCatalog.RandomizedPca(Microsoft.ML.AnomalyDetectionCatalog.AnomalyDetectionTrainers,Microsoft.ML.Trainers.RandomizedPcaTrainer.Options)"/>
58+
/// <seealso cref="PcaCatalog.RandomizedPca(AnomalyDetectionCatalog.AnomalyDetectionTrainers, string, string, int, int, bool, int?)"/>
59+
/// <seealso cref="PcaCatalog.RandomizedPca(AnomalyDetectionCatalog.AnomalyDetectionTrainers, RandomizedPcaTrainer.Options)"/>
6060
/// <seealso cref="Options"/>
6161
public sealed class RandomizedPcaTrainer : TrainerEstimatorBase<AnomalyPredictionTransformer<PcaModelParameters>, PcaModelParameters>
6262
{

0 commit comments

Comments
 (0)