-
Notifications
You must be signed in to change notification settings - Fork 0
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
add violin plots fct #57
Conversation
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 put a few suggestions, but it looks good :)
plot_kw : dict, optional | ||
Arguments to pass to the `seaborn.violinplot()` function. | ||
color : str, int or list, optional | ||
Unique color or list of colors to use. Integers point to the applied stylesheet's colors, in zero-indexed order. |
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 like that we can access the colors of the stylesheet!
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.
It would be nice to show them somewhere similarly to what you did in the color association section of the notebook. maybe in the stylesheet section of the notebook ?
|
||
# set default use_attrs values | ||
if "orient" in plot_kw and plot_kw["orient"] == "h": | ||
use_attrs = {"xlabel": "long_name", "xunits": "units"} | use_attrs |
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 am wondering if we want vertical gridline when the violons are horizontal ?
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.
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.
Looks good!
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.
Super, est-ce que les gridlines verticales seront ajoutées automatiquement si l'utilisateur met l'option de violions horizontaux?
oui je l'ai ajouté dans ma branche! (pas encore push) |
Added a
violin()
function and examples in docs.It wraps the seaborn function pretty thinly, but it:
use_attrs
.I went with
violin()
instead ofviolinplot()
to not exactly mirror the seaborn name, but it can be modified.