Lockdown Microsoft.ML.PCA public surface

[ganik]

fixes #2275

[Ivanidzo4ka]

    public const string LoaderSignature = "PcaTransform";

internal

---

Refers to: src/Microsoft.ML.PCA/PcaTransformer.cs:227 in 5ac94da. [](commit_id = 5ac94da, deletion_comment = False)

[Ivanidzo4ka]

PublicKey [](start = 67, length = 9)

if you change PcaTests.cs to use ml.Transforms.Projection.ProjectToPrincipalComponents instead of direct estimator call would you still need this change?

[sfilipi]

    public class Column : OneToOneColumn

internal

---

Refers to: src/Microsoft.ML.PCA/PcaTransformer.cs:60 in b54ce70. [](commit_id = b54ce70, deletion_comment = False)

[sfilipi]

        public readonly int? Seed;

one line XML

can borrow from below.

---

Refers to: src/Microsoft.ML.PCA/PcaTransformer.cs:107 in b54ce70. [](commit_id = b54ce70, deletion_comment = False)

[sfilipi]

    public sealed class ColumnInfo

one line XML

---

Refers to: src/Microsoft.ML.PCA/PcaTransformer.cs:99 in b54ce70. [](commit_id = b54ce70, deletion_comment = False)

[sfilipi]

    public sealed class ColumnInfo

move to estimator.

---

Refers to: src/Microsoft.ML.PCA/PcaTransformer.cs:99 in b54ce70. [](commit_id = b54ce70, deletion_comment = False)

[Ivanidzo4ka]

I think @artidoro already done all this work in his PR #2366 maybe it's better to collaborate with him and close this PR?

[ganik]

right, will ask @artidoro to assign project13 issues onto himself to avoid confusion

---

In reply to: 459872486 [](ancestors = 459872486)