X Tutup
Skip to content

ENH: add unified seaborn style sheet#6467

Merged
WeatherGod merged 1 commit intomatplotlib:masterfrom
tacaswell:sty_single_sns_sheet
May 24, 2016
Merged

ENH: add unified seaborn style sheet#6467
WeatherGod merged 1 commit intomatplotlib:masterfrom
tacaswell:sty_single_sns_sheet

Conversation

@tacaswell
Copy link
Copy Markdown
Member

closes #6452

attn @mwaskom I think this is half of what you had in mind.

@mwaskom
Copy link
Copy Markdown

mwaskom commented May 23, 2016

LGTM, thanks!

@efiring
Copy link
Copy Markdown
Member

efiring commented May 23, 2016

Is there a reason why it is called "sns" instead of "seaborn"?

@tacaswell
Copy link
Copy Markdown
Member Author

sns is the normal import alias of seaborn and is less letters.

@tacaswell tacaswell added this to the 2.0 (style change major release) milestone May 23, 2016
@WeatherGod
Copy link
Copy Markdown
Member

As someone who doesn't use seaborn, I wouldn't have guessed that "sns" was
the seaborn style.

On Sun, May 22, 2016 at 10:00 PM, Thomas A Caswell <notifications@github.com

wrote:

sns is the normal import alias of seaborn and is less letters.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#6467 (comment)

@tacaswell tacaswell force-pushed the sty_single_sns_sheet branch from dfcca21 to 6d5a586 Compare May 23, 2016 19:14
@tacaswell
Copy link
Copy Markdown
Member Author

renamed to seaborn

@WeatherGod WeatherGod merged commit 8bab42d into matplotlib:master May 24, 2016
@WeatherGod
Copy link
Copy Markdown
Member

this need to be backported at all?

@efiring
Copy link
Copy Markdown
Member

efiring commented May 24, 2016

Yes, it should go to v2.x.

WeatherGod added a commit that referenced this pull request May 24, 2016
ENH: add unified seaborn style sheet
@WeatherGod
Copy link
Copy Markdown
Member

backported to v2.x via e70bc42

@tacaswell tacaswell deleted the sty_single_sns_sheet branch July 9, 2016 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Please add a generic "seaborn" style

5 participants

X Tutup