Skip to content
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

Colormap choice guidelines in documentation - based on talk at SciPy 2014 #3301

Merged
merged 6 commits into from
Jul 29, 2014

Conversation

kthyng
Copy link
Contributor

@kthyng kthyng commented Jul 25, 2014

Finally got something together for colormaps documentation, @tacaswell and @dmcdougall!

@kthyng kthyng mentioned this pull request Jul 25, 2014
@tacaswell tacaswell added this to the v1.4.0 milestone Jul 26, 2014
@mwaskom
Copy link

mwaskom commented Jul 26, 2014

Just dropping in to say that this looks fantastic!

@tacaswell
Copy link
Member

Would it be possible to get the figures too?

@kthyng
Copy link
Contributor Author

kthyng commented Jul 28, 2014

@tacaswell Ah, I forgot to include the scripts before. Oops! They are there now and the build has passed. How is it now?

tacaswell added a commit that referenced this pull request Jul 29, 2014
DOC : added colormap information

Colormap choice guidelines in documentation - based on talk at SciPy 2014

Only changes to rst + examples scripts.
@tacaswell tacaswell merged commit 572731f into matplotlib:v1.4.x Jul 29, 2014
@tacaswell
Copy link
Member

Great! This builds cleanly with the figures on my system. I am going to go ahead and merge this, we can always fine-tune the prose later.

@kthyng
Copy link
Contributor Author

kthyng commented Jul 29, 2014

Awesome! Yep changes are fine. I just wanted to get it out.

On Mon, Jul 28, 2014 at 10:32 PM, Thomas A Caswell <notifications@github.com

wrote:

Great! This builds cleanly with the figures on my system. I am going to go
ahead and merge this, we can always fine-tune the prose later.


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

Kristen M. Thyng
Postdoctoral Research Associate
Department of Oceanography
Texas A&M University
http://kristenthyng.com

@dmcdougall
Copy link
Member

@kthyng Congratulations, you're now a contributor!

@NelleV
Copy link
Member

NelleV commented Jul 29, 2014

I'm very happy to see this in. I've been struggling to understand a plot from a paper this morning, and it turns out the gray scale printed version I was looking at was very misleading (jet colormap!).

@kthyng
Copy link
Contributor Author

kthyng commented Jul 30, 2014

@dmcdougall Thanks! I knew something felt different...

@NelleV Jet strikes again! I hope this stuff can help some people along the way.

@WeatherGod
Copy link
Member

My advisors one time wondered why it was so "difficult" for me to create a
grayscale version of my radar images for a publication (it was going to be
color version for the electronic copy, but grayscale for the print copy).
"Just run the color version through Photoshop!"

Sigh...

On Wed, Jul 30, 2014 at 11:35 AM, Kristen Thyng notifications@github.com
wrote:

@dmcdougall https://github.com/dmcdougall Thanks! I knew something felt
different...

@NelleV https://github.com/NelleV Jet strikes again! I hope this stuff
can help some people along the way.


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

@dmcdougall
Copy link
Member

@WeatherGod 85a26dc

:)

@j9ac9k
Copy link

j9ac9k commented Mar 10, 2023

@kthyng Just wanted to post and relay a huge thank you for creating these docs. My use case was a bit different (not plotting inside matplotlib, but I have access to matplotlib colormap objects) and your talks got me 90% of the way done, thank you so much!!!

@kthyng
Copy link
Contributor Author

kthyng commented Mar 10, 2023

Awesome! Glad to help.

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.

None yet

7 participants