-
-
Notifications
You must be signed in to change notification settings - Fork 113
Expose windbarbs #1711
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?
Expose windbarbs #1711
Conversation
maximlt
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 haven't yet run the code yet)
My mental model is that geo=True (or any of the geo-related options except tiles) == geoviews is required. That's a small deviation from that, which I guess is fine but I wanted to point it out.
Have you considered calling it windbarbs? Even if matplotlib calls it barbs, I think windbarbs would be my preference. Right not it sits alphabetically between bar and barh which is unfortunate, and shows up as one of the first suggestions on autocomplete, while it's a niche functionality.
Closes #1695
With barbs in geoviews bokeh backend and also available on matplotlib backend, we should expose barbs on hvplot.
I think I covered all the docs references, but I could be missing some.