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

WIP: Add new Colormaps to docs #4965

Merged
merged 5 commits into from Oct 21, 2015

Conversation

jenshnielsen
Copy link
Member

First pass at a fix for #4783

The new colormaps have been added to the relevant examples and the plots in the user guide and tweaked the layout a bit.

So far I have just added the colormaps. We should probably promote the new ones better?

The plots in the user guide are still in LAB space I guess we should convert them to CAM02-UCS?

We should probably alto mention viscm and @njsmith's talk from Scipy

@stefanv @njsmith and @kthyng you may have some interest in this.

'gist_heat', 'gray', 'hot', 'pink',
'spring', 'summer', 'winter']),
'gist_heat', 'gray', 'hot', 'inferno', 'magma',
'pink', 'plasma', 'spring', 'summer', 'viridis',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe pull the 4 new ones out into their own category 'Suggested Sequential' ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, Im away until Wednesday but will try to finish it after. Perhaps we should add some more metadata to the colormaps. This info is duplicated 3 or 4 times in various examples.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By Suggested Sequential you mean we suggest them because they have perceptual uniformity.

In which case why not call them Perceptually Uniform Sequential. That way we make it clear why we suggest these ones, i.e. why they should use these ones over the other sequential colour maps...

@mdboom
Copy link
Member

mdboom commented Oct 20, 2015

@jenshnielsen: What do you think about adding #5202 to this? If you don't have time, I can take a crack at this.

@jenshnielsen
Copy link
Member Author

Makes sense to me. If you have time to work on it that would be great. I am not likely to have time to look into it again before the weekend

@mdboom mdboom self-assigned this Oct 20, 2015

- Whether representing form or metric data ([Ware]_)
- Your knowledge of the data set (*e.g.*, is there a critical value from which the other values deviate?)
- If there is an intuitive color scheme for the parameter you are plotting
- If there is a standard in the field the audience may be expecting

For many applications, a perceptual colormap is the best choice --- one in which equal steps in data are perceived as equal steps in the color space. Researchers have found that the human brain perceives changes in the lightness parameter as changes in the data much better than, for example, changes in hue. Therefore, colormaps which have monotonically increasing lightness through the colormap will be better interpreted by the viewer.
For many applications, a perceptual colormap is the best choice --- one in which
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perceptually uniform colormap

@mdboom mdboom mentioned this pull request Oct 20, 2015
@jenshnielsen jenshnielsen merged commit 38b8911 into matplotlib:master Oct 21, 2015
@jenshnielsen
Copy link
Member Author

Replaced by #5284

@WeatherGod
Copy link
Member

Did you mean to merge or close?

On Wed, Oct 21, 2015 at 5:00 PM, Jens Hedegaard Nielsen <
notifications@github.com> wrote:

Replaced by #5284 #5284


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

@QuLogic
Copy link
Member

QuLogic commented Oct 21, 2015

This PR was automatically marked as merged by merging #5284 since it was based on this one (without any rebasing.)

@jenshnielsen
Copy link
Member Author

@QuLogic is right all the commits in this one were on #5284 too

@OceanWolf
Copy link
Contributor

clever gitty

@jenshnielsen jenshnielsen deleted the colormapsrefs branch November 9, 2015 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants