ML Notebooks should use Plotly.NET

dotnet / csharp-notebooks

Get started learning C# with C# notebooks powered by .NET Interactive and VS Code.

MIT License

1.03k stars 344 forks source link

ML Notebooks should use Plotly.NET #44

Closed beccamc closed 2 years ago

beccamc commented 2 years ago

Replace XPlot with Plotly.NET.

Blocked on..

Plotly.NET getting signed. https://github.com/plotly/Plotly.NET/issues/175
Also waiting on C# improvements https://github.com/plotly/Plotly.NET/issues/285

kMutagene commented 2 years ago

Both should be resolved: The newest releases have strong names and basic native C# bindings are available as well (https://github.com/plotly/Plotly.NET/pull/294, https://www.nuget.org/packages/Plotly.NET.CSharp/).

JakeRadMSFT commented 2 years ago

@kMutagene thoughts on the interactive package depending on the CSharp package?

kMutagene commented 2 years ago

@JakeRadMSFT what would be the benefit of that other than only referencing one package in C# notebooks. Ideally I would like to keep these packages separated, as that would also mean that an F# notebook using the interactive package would install an unused dependency.

JakeRadMSFT commented 2 years ago

Resolved with: https://github.com/dotnet/csharp-notebooks/pull/49