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

Colorbars for variables on pressure levels #1139

Closed
jwarner8 opened this issue Feb 12, 2025 · 1 comment · Fixed by #1140
Closed

Colorbars for variables on pressure levels #1139

jwarner8 opened this issue Feb 12, 2025 · 1 comment · Fixed by #1140
Assignees
Labels
enhancement New feature or request

Comments

@jwarner8
Copy link
Contributor

jwarner8 commented Feb 12, 2025

Currently, there are no entries for pressure level variables. If we set a variable vmin, vmax, for say geopotential height, we would need a huge range to cater for this variable on 200hPa vs 850hPa, and so the users wouldn't be able to identify any features. Currently the colorscale automatically scales to each plot, so can change as the user scrolls through time. I thought about potential solutions such as expanding the colorbar read for cmap, levels, norm = _colorbar_map_levels(cube.name(),level=int(cube.coord('pressure').points[0])), but we would end up with a huge list in colorbar for every pressure level that a user might choose.

@Sylviabohnenstengel
Copy link
Contributor

Sylviabohnenstengel commented Feb 12, 2025

By coincidence have implemented something along those lines in #1140. but this would blow up the json file :-(

Maybe we need to think around interpolation for the colourbar.

@Sylviabohnenstengel Sylviabohnenstengel self-assigned this Feb 12, 2025
jfrost-mo pushed a commit that referenced this issue Feb 14, 2025
Only includes the colourbar definitions for temperature. More will be
added later.

Fixes #1139
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants