-
-
Notifications
You must be signed in to change notification settings - Fork 113
Roadmap updates #1694
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
base: main
Are you sure you want to change the base?
Roadmap updates #1694
Conversation
hoxbro
left a comment
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.
Some very minor comments.
| ### 2. Refactor the test suite | ||
|
|
||
| hvPlot users have repeatedly complained about the quality of its documentation, which seems to mostly stem from: | ||
| The test suite should be re-factored and improved to catch bugs before users have to report them and to prevent potential regressions. |
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.
Would you like to add testing for cuDF, even if only to run locally?
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.
That's something I'm gonna do but would be happy if you want to do it. I don't think there's a need to add it to the roadmap though.
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.
This was the answer I hoped for 👍
And I also don't think it should be on the roadmap, but it's more of a curiosity for me.
ahuang11
left a comment
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.
I'm fine with all, and would like to add an additional comment
5. Increase hvPlot's presence within PyData
Since we're releasing 1.0, I think it may be worth revisiting hvPlot’s default settings to elevate the out-of-the-box UI/UX and follow some best data visualization practices from books like Storytelling Data Visualization and/or How Charts Lie, or at least document how to do it in hvPlot.
Many users reach for Plotly or Seaborn because their defaults are immediately more visually appealing (and in Seaborn’s case, offer built-in statistical plotting). We could also document how to better integrate with websites, dashboards, and publications.
Co-authored-by: Simon Høxbro Hansen <[email protected]>
These are all great ideas! I'll personally focus on the items I listed (if they're fine with other maintainers), so if you want your suggestions to be part of the 1.0 release, it's likely you'll have to do the work yourself, including reaching consensus on the new UI/UX defaults (which should be possible). |
|
The visually appealing part, I think it's a matter of changing up the themes as the default, which shouldn't be a huge lift, e.g. https://medium.com/@pYdeas/create-publication-styled-figures-with-hvplot-using-the-bokeh-backend-b623ab48a5d2 Perhaps adding more themes to Bokeh, and then documenting how to use themes in hvplot, which currently has no documentation on (https://hvplot.holoviz.org/en/docs/latest/search.html?q=theme). Also pretty sure AI can help with it as well :) Did you want me to edit the roadmap with these new desired changes, or wait for others to comment? For the books, I think I need to re-review what's in them. One of them I just remember is ensuring the legends ordering semi-match with the lines' min to max, or simply adding labels directly to the lines, like this:
If we can add some easy way for users to do this, like hvplot ui/ux util methods to do this, I think that'd be great. |

We need approval from all maintainers.