-
-
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 <simon.hansen@me.com>
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. |
Technically indeed I guess that's not a big lift. Coming up with a set of defaults we all agree on will be the challenge for you.
Having more Bokeh themes sounds good, especially if they're well-maintained and thoroughly developed. In the Python viz landscape there are already well-known themes that I'm sure would be interesting to make available in Bokeh (stock matplotlib, seaborn, plotly, ggplot, etc.), which in turn would make them available in HoloViews and hvPlot. Creating a new theme from scratch sounds more challenging. Adding documentation on how to set a theme in hvPlot sounds like a great idea, I think that would be a how-to.
Well I'm okay with that being part of the 1.0 roadmap but I'd suggest you create first a separate issue and add more details about what you intend to do before the 1.0 release. I've already discussed multiple times the other items (docs, test suite refactor, pie charts, narwhals) so there was already some sort of agreement.
I'm not sure what you mean with util methods. Probably worth being discussed in another issue. |
philippjfr
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 started commenting on individual items but considering the document as a whole I think my main problem with it is that there's some conflation of specific goals and general principles that we're aiming for. In other words, a lot of it doesn't necessarily read like a roadmap but general statements of intent. Which I think is actually perfectly appropriate but should imo not be folded in with the more specific items talking about specific features.
The essence I'm getting from a number of the items is that the focus is moving away from just adding more features to stability by improving test coverage, fixing bugs, tracking HoloViews feature development, increasing adoption etc. and then there are more specific and classic roadmap items.
My suggestion here would be to start out with a paragraph outlining the overall strategy, i.e. the why, the focus on stability, testing, bug fixes, adoption etc. and only then breaking down specific quantifiable goals for both pre- and post 1.0 releases.

We need approval from all maintainers.