-
-
Notifications
You must be signed in to change notification settings - Fork 10
Convert Plotly to VTK.js and remove dist folder #64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
9161044
2335be6
b4cd9f5
b30a882
46991c2
d4447b2
89643f2
f8927a9
9d191aa
b052997
6d5f966
bf1e0b5
434ef04
c509948
4673a26
ac8fde6
aad3620
aa7dc0f
d0245f7
7a5e44c
8342666
403bc2c
2bd2f5c
177b82c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
This file was deleted.
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
README now instructs users to install only
@kitware/vtk.js, but the codebase still supports a Plotly backend (renderOptions.backend === "plotly") andplotly.jsremains an (optional) peer dependency. Consider either documenting Plotly as an optional install (and how to select the backend) or removing Plotly support/dependencies to match the migration messaging.