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

Center diverging colormaps at 0 #518

Open
jwdebelius opened this issue May 26, 2021 · 3 comments · May be fixed by #521
Open

Center diverging colormaps at 0 #518

jwdebelius opened this issue May 26, 2021 · 3 comments · May be fixed by #521

Comments

@jwdebelius
Copy link

Hi Empress team,

I'm enjoying this a lot so far!

Right now, I have a colormap between -1.49 to 1.69, centered at 0.1. Which is okay-ish, but slightly visually confusing.

Im wondering if it would be possible here (an possibly in emperor) to center diverging colormaps (maybe a as a check box?). I think it is more intuitive that positive values are all one color (say red) and negative values are all another (blue), with equal color weight.

The proposed behavior would take 0-centered colormap, which I usually approach in other settings by finding find the most extreme upper/lower value, and set the colormap to reflect that.

Thanks!
Justine

@gibsramen
Copy link
Collaborator

Hi, @jwdebelius

This is definitely on our radar - see #365 (comment). Would you prefer your proposed checkbox approach to specifying numerical values for end/midpoints?

@jwdebelius
Copy link
Author

Hi @gibsramen,

I'm glad to know I'm not the only one. I really like the colorbrewer colormap, so the ability to say show RdBu scaled between -1.75 and 1.75 would kind of be my idea. That automatically sets my central value, and lets me line up with colormaps that might exist elsewhere.

@fedarko
Copy link
Collaborator

fedarko commented May 27, 2021

Agreed -- it would be great to have this in. I'll see if we can get it added in soon :)

@kwcantrell kwcantrell linked a pull request May 29, 2021 that will close this issue
@kwcantrell kwcantrell linked a pull request Jun 1, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants