Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add Seaborn style sheets; addresses #4566#4570

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
tacaswell merged 2 commits intomatplotlib:masterfrompetebachant:seaborn_style
Jul 5, 2015

Conversation

petebachant
Copy link
Contributor

This commit adds the default Seaborn "darkgrid" style to the style library.

@efiring
Copy link
Member

Thanks for doing this.
In anticipation of additional Seaborn styles being added, should the file name include some indication of the particular variant?
Maybe we need some more thought about how to categorize, organize, and name styles.

@tacaswelltacaswell added this to thenext point release milestoneJun 30, 2015
@petebachant
Copy link
ContributorAuthor

I was thinking of calling this one simplyseaborn since it is the default style when importingseaborn, then addingseaborn-white,seaborn-whitegrid, andseaborn-ticks.

@efiring
Copy link
Member

Attn@mwaskom: comments?

@mwaskom
Copy link

I would prefer the current default seaborn style to be calledseaborn-darkgrid (or however you choose to indicate the variant).

Do matplotlib stylesheets have any facility for handling style and element scale ("context" in seaborn, regrettably) independently?

@WeatherGod
Copy link
Member

There is MEP26 that is being ironed out, if that is what you are thinking?
http://matplotlib.org/devdocs/devel/MEP/MEP26.html

On Tue, Jun 30, 2015 at 3:59 PM, Michael Waskomnotifications@github.com
wrote:

I would prefer the current default seaborn style to be called
seaborn-darkgrid (or however you choose to indicate the variant).

Do matplotlib stylesheets have any facility for handling style and element
scale ("context" in seaborn, regrettably) independently?


Reply to this email directly or view it on GitHub
#4570 (comment)
.

@mwaskom
Copy link

I just mean that the "seaborn styles" are really the product of two independent sets of rc parameters.

@petebachant
Copy link
ContributorAuthor

The scaling and colors could be split into two different stylesheets, which would allow users to

plt.style.use("seaborn-darkgrid")plt.style.use("seaborn-paper")

@mwaskom
Copy link

Actually good point, you also need a third piece: the color cycle palette.

@tacaswell
Copy link
Member

Would it be worth adopting the scale style sheets wholesale so

plt.style.use('seaborn-darkgrid")plt.style.use("media-paper")# plt.style.use("media-interactive")# plt.style.use("media-talk")

?

The color cycle might be complicated / should embrace#4258 (which needs to be replaced by a PR to addhttps://github.com/tacaswell/cycler (which needs to end up on pypi and move to the mpl org) ).

@mwaskom
Copy link

I don't think seaborn does anything as complicated as your cycler tool, it just sets the color cycle independently from the axes style and plotting context.

@tacaswell
Copy link
Member

Right, leave cycler out of this.

@petebachant Can you split the style sheet up as discussed (base axes settings, font/tick sizes, color cycle)?

There is a PR in to use styles in styles (#4240 ) which can be used to create composite style sheets if we want that as well.

@petebachant
Copy link
ContributorAuthor

Sure, I will split the style sheets up.

@petebachantpetebachant changed the titleAdd Seaborn stylesheet; addresses #4566Add Seaborn style sheets; addresses #4566Jul 4, 2015
@petebachant
Copy link
ContributorAuthor

Just split the darkgrid style into three sheets for the scaling, colors, and palette. What do you guys think? Should I make the other Seaborn variants?

@tacaswell
Copy link
Member

👍 to both this and the other variants.

@tacaswell
Copy link
Member

@petebachant I am going to merge this as-is, can you make a new PR with the other variants ?

tacaswell added a commit that referenced this pull requestJul 5, 2015
@tacaswelltacaswell merged commit0bbd127 intomatplotlib:masterJul 5, 2015
@petebachant
Copy link
ContributorAuthor

Will do!

@petebachantpetebachant deleted the seaborn_style branchJuly 7, 2015 13:24
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.5.0
Development

Successfully merging this pull request may close these issues.

5 participants
@petebachant@efiring@mwaskom@WeatherGod@tacaswell

[8]ページ先頭

©2009-2025 Movatter.jp