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

Deprecate panel.pane.Vizzu proposal #7471

Open
ahuang11 opened this issue Nov 7, 2024 · 5 comments
Open

Deprecate panel.pane.Vizzu proposal #7471

ahuang11 opened this issue Nov 7, 2024 · 5 comments
Labels
deprecation A proposal to deprecate a feature

Comments

@ahuang11
Copy link
Contributor

ahuang11 commented Nov 7, 2024

Feature Overview
panel.pane.Vizzu integrates Vizzu for animated and interactive visualizations. However, as noted in PR #7356, it increases Panel's complexity and size, particularly affecting environments like Pyodide.

Proposal
Deprecate panel.pane.Vizzu and migrate it to a standalone extension using the panel-copier-template (reference PR). This separation will streamline Panel’s core and allow for easier community maintenance.

Impact
Usage seems limited; there are a few feature requests:
Allow Vizzu stories in Panel
Embed Vizzu plots in static HTML
Add an IpyVizzu pane

Effort

  • Create a new extension using the copier template.
  • Migrate code and update documentation.
  • Add deprecation warnings in Panel.
  • Announce the change to encourage community contributions.

This can also be extended to a few other panes, like VTK, deckgl, and echarts, etc

@philippjfr
Copy link
Member

What I'm missing from this proposal is an explanation why Vizzu in particular should be deprecated? In general I like having support for a wide range of visualization libraries in Panel core. For VTK I think the argument is very clear because it is quite a complex piece to maintain and support with a lot of code behind it. For deck.gl, echarts, Vizzu I think the case is much less strong.

@ahuang11
Copy link
Contributor Author

ahuang11 commented Nov 7, 2024

To me, I think it's a matter of popularity/usage, which is why I think Vizzu could be deprecated. Matplotlib is one of the most popular libraries so I think the case is clear that it should be kept, as well as HoloViews, which HoloViz maintainers maintain, and Bokeh is a dependency.

For deck.gl, echarts, Vizzu I think the case is much less strong.

I'm not terribly familiar with Vizzu, but I think Panel only supports parts of Vizzu? So when Vizzu support gets expanded on, like for #6216, the lines of code will increase? And from my experience, deck.gl support was also limited.

Finally, I think if it gets separated out into another repo, ideally, there'd be someone interested in claiming ownership of it and maintaining it (ideally), similar to all the community contributed extensions in streamlit https://streamlit.io/components

@MarcSkovMadsen
Copy link
Collaborator

MarcSkovMadsen commented Nov 7, 2024

To me it would be a good idea to carve out Vizzu.

  • I don't see data indicating its being used.
  • its not being kept up to date. At least the last time I tried using it there was a big difference in api between current Vizzu JS and our Vizzu pane. And I believe I had to put in a PR to explain in the reference guide which version of Vizzu JS docs users should use. Otherwise they would not be able to use it at all.
  • Personally I don't understand the api choices. I would have expected it to be close to IPyVizzu but I cannot see that. I believe it makes it harder to start using than it has to be.

I think having it live in a separate repository is fine. If there starts being an interest it can develop more quickly and making breaking changes to improve the api would also be fine.

What is in core Panel should be of high quality.

@philippjfr philippjfr added the deprecation A proposal to deprecate a feature label Nov 14, 2024
@maximlt
Copy link
Member

maximlt commented Nov 15, 2024

Looking at the pageview stats, the Vizzu pane has about 100 hits last month, a little more than Audio/SVG and less than JSON/PNG.
Image

For reference, the most visited ones had about 500-600 hits last month.

Image

Interestingly, the most visited panes are plotting related (bokeh, plotly, holoviews).


I'd like to know how popular Vizzu is in the JavaScript ecosystem?

@simzer
Copy link
Contributor

simzer commented Nov 15, 2024

its not being kept up to date.

Vizzu has a maintained Python interface called IpyVizzu. It would make sense if the panel integration used it instead of providing a separate implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecation A proposal to deprecate a feature
Projects
None yet
Development

No branches or pull requests

5 participants