Skip to content
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

remove holoviews dependency #8

Open
chanana opened this issue Jun 27, 2023 · 2 comments
Open

remove holoviews dependency #8

chanana opened this issue Jun 27, 2023 · 2 comments

Comments

@chanana
Copy link
Collaborator

chanana commented Jun 27, 2023

I don't love it

@jtprince
Copy link
Collaborator

jtprince commented Jul 6, 2023

Yeah, if the abstraction level of holoviews is not right, then it makes sense to drop it down one notch.

@chanana
Copy link
Collaborator Author

chanana commented Jul 6, 2023

it's really cool as a concept but in practice there's things that become enough of an edge case that we could drop it potentially.

  1. you can save to svg if you have a matplotlib or bokeh plot but not a plotly plot. However, plotly shows you how to save the svg separately using the kaleido package.
  2. you can set an image size if you're using plotly or bokeh but not if you're using matplotlib. Obviously, matplotlib on its own does have that capability.
  3. plotting a scatter plot as Scatter vs Points has subtleties that I need to get acquainted with (in holoviews) and can result in differences in how you interact with the plot for the different engines

These were the things I ran into so far. The other solution here would be to attempt to put PRs upstream in holoviews itself so we can use it more consistently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants