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

Add Spectral-like colormaps based on pyart#1320 #1325

Merged
merged 2 commits into from
Nov 16, 2022

Conversation

jthielen
Copy link
Contributor

This PR implements a version of the ChaseSpectral colormap described in #1320 (configured with -10 dBZ as minimum and 80 dBZ as maximum, so that highest lightness occurs at roughly the midpoint of ~35 dBZ).

image

I also played around with some parameter smoothing and further lightness adjustments in CIELab space, and ended up with something that looks like matplotlib's "Spectral_r" colormap extended with "magma_r" on the high-end and a sequential-from-black colormap on the low-end. I called it SpectralExtended, and included it here as well.

image

@wx4stg
Copy link
Contributor

wx4stg commented Nov 10, 2022

love it! this weekend I'll be looking into converting these to GR/radarscope formats...

@zssherman
Copy link
Collaborator

Thanks @jthielen for putting this together! On the code side of things this looks good! I approved for the test suite to run, @mgrover1, we might want to consider in the future moving the .txt files to a dedicated directory.

Copy link
Collaborator

@mgrover1 mgrover1 left a comment

Choose a reason for hiding this comment

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

One small suggestion - @jthielen can you add that new colormap to the example in the gallery?

Copy link
Collaborator

@mgrover1 mgrover1 left a comment

Choose a reason for hiding this comment

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

Great! Thanks @jthielen !

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.

4 participants