Separate assemblies for static extensions

[abgoswam]

Fixes #1695

• Created separate assemblies to host the Static API extensions. Moved the Static API extensions to the appropriate assembly.
• Requires adding [BestFriend] attribute in several places

1. Microsoft.ML.StaticPipe
2. Microsoft.ML.LightGBM.StaticPipe
3. Microsoft.ML.TensorFlow.StaticPipe
4. Microsoft.ML.OnnxTransform.StaticPipe
5. Microsoft.ML.HalLearners.StaticPipe

NOTE: These assemblies are named corresponding to nuget packages (with the Microsoft.ML prefix)

[abgoswam]

@Zruty0 . I have created the assemblies (to host static API extensions) so they correspond to nuget packages having the Microsoft.ML prefix. It would be great if you could take a look.

[sfilipi]

public [](start = 4, length = 6)

ideally would want to document, but i get this is low-pri. #Resolved

[abgoswam]

yeap. low-pri .

not looking to modify documentation in this PR

---

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

[sfilipi]

        var invalidData = TextLoader.CreateReader(Env, ctx => (

I think this one should get moved to StaticPipe too. This is a bit trickier: it is a partial class and therefore on the same namespace as the Text Loader. #Closed

---

Refers to: test/Microsoft.ML.Tests/Transformers/TextFeaturizerTests.cs:37 in a94d7f4. [](commit_id = a94d7f4, deletion_comment = False)

[abgoswam]

        var invalidData = TextLoader.CreateReader(Env, ctx => (

I am thinking of keeping the TextLoader changes as a separate PR. As you noted, it seems a bit more involved

This PR is for the transforms / learners

---

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

---

Refers to: test/Microsoft.ML.Tests/Transformers/TextFeaturizerTests.cs:37 in a94d7f4. [](commit_id = a94d7f4, deletion_comment = False)

[Zruty0]

using Microsoft.ML.Core.Data;

Please make sure all the new files have header comments #Resolved

---

Refers to: src/Microsoft.ML.StaticPipe/WordEmbeddingsStaticExtensions.cs:1 in a94d7f4. [](commit_id = a94d7f4, deletion_comment = False)

[Zruty0]

using [](start = 0, length = 5)

sort usings #Resolved

[Zruty0]

Looks good to me with minor comments.

I noticed that we don't yet create NuGet packages for the new static pipe assemblies. Should we do it now (maybe even in this PR), or wait? @TomFinley do you have an opinion?

---

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

[Zruty0]

[sfilipi]