-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Please add a generic "seaborn" style #6452
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
Comments
Did that PR to put styles in style sheets get merged? |
Not sure, but I just came across this which prompted the issue. |
Here is the PR that originally added the seaborn style sheets along with I have no problem with a unified seaborn style sheet, but right now, I On Thu, May 19, 2016 at 12:13 PM, Thomas A Caswell <notifications@github.com
|
But you could just put all of the aesthetics into one sheet, right? |
correct On Thu, May 19, 2016 at 12:18 PM, Michael Waskom notifications@github.com
|
Hah, I see now that I initially advocated for what I'm now objecting to... Although I think what I probably had in mind was that Anyway, seeing the result of my hasty suggestion, I do think that it's not ideal because users who want their plots to look "like seaborn" should probably be able to do so with minimal effort. Currently you'd have to be aware of a number of relatively internal seaborn details (like the bad name for the default palette). I also think that having so many "seaborn" styles is likely very confusing in the context of tools like this, because they totally dominate the list. Ultimately what to do about that is up to you folks, but I'd suggest that it might be better to scale back the extent to which matplotlib styles fully replicate the aesthetic control that seaborn has. (Similarly, the "ggplot" style is just one of the many themes that ggplot can do). |
I would be completely in favor of putting in a consolidated "seaborn" style sheet. As for the other sheets, maybe a way to specify "sub-sheets" might help, perhaps? Is there any analogous concept in CSS? |
I am 👍 on adding a single 'seaborn' sheet, but am a bit worried about removing styles that we have been shipping for a year. |
Thanks @tacaswell ! |
I'm glad that matplotlib has included seaborn aesthetics in its built-in style sheets. However, the way that the different elements of the seaborn aesthetics were split up (i.e., that there are separate
seaborn-darkgrid
, seaborn-deep`, etc.) is both a little confusing and produces results that are, imo, visually unappealing.I would much prefer if there were also a single "seaborn" style that encompasses
seaborn-darkgrid
,seaborn-deep
, andseaborn-notebook
, so that doingplt.style.use("seaborn")
would create plots that look the same as actually importing seaborn.The text was updated successfully, but these errors were encountered: