-
Notifications
You must be signed in to change notification settings - Fork 6
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
Review colormap legend functionality #929
Comments
Assigning this to myself, @j08lue |
Related: WHIS recently enabled the changing of rescale and colormap values: https://water-health-ind.netlify.app/ |
Spoke with @danielfdsilva about the WHIS implementation:
I also asked on the veda-data-services slack channel. I will report back |
The colormap values come from matplotlib https://matplotlib.org/stable/gallery/color/colormap_reference.html |
The fact that Rainbow is the top selected ramp right now might actually be a problem, and might need to be replaced with https://research.google/blog/turbo-an-improved-rainbow-colormap-for-visualization/ (called turbo in the R link above). Of course this also all depends on the default background layer used. There's a lot of prior art in this domain, but we can probably drop some of the ramps based by preferencing ramps that are better at larger value differences (i.e. greens are great because humans can differentiate a lot of greens). I'll reference back to https://colorbrewer2.org/ for helping decide which of the 1-3 color variants to pick from. |
@slesaad mentioned today that there might be some work on data services to enable to selection of different color maps. @sandrahoang686 might be worth a convo in the upcoming team sync |
@faustoperez Deborah asked a question about this today
I assume that is what would be indicated by "Default," is that correct? If so, did you consider other words, like, "Preferred" or something along those lines? I'm a bit ambivalent on which word we use, just wanted to pass along the question and thought. Also, might be valuable to show the state once somebody selects another option (not default) of colormap |
@aboydnw yes the 'default` as defined in the STAC Render extension should be considered the Preferred by data provider. |
@wildintellect what would that look like? The default/preferred should be set by the data provider during configuration |
Re configuration workflow
Not sure this is what you meant, @wildintellect, but I could see a very nice data configuration workflow for data providers like this:
But this is not VEDA UI material, just ideas for STAC Admin etc. development (cc @oliverroick, @ricardoduplos) - inspiration Mapbox Studio |
Re TiTiler upgrade: |
Thanks all for the great feedback!
We'll keep the histogram functionality out for now 👍
Categorical layers won't have a colormap selector and will default to the preferred colors from the data provider
Totally agree. I like the suggestion of offering Turbo as a better option on the selector, but not using rainbow palettes is something that has to come from the data providers in the configuration phase.
This piece of work relates to the E&A page so the layer would already be loaded on the map. Happy to work on a tool to help data providers choose ramps during the configuration process so we can introduce better practices. This could also include the basemap selection 🤔
The preferred colormap from the provider is at the top of the list and labeled as "Default". I will work on the designs to make it more evident, and show the state when other ramp is selected. |
Btw, we already have a categorical dataset in EIC and Earthdata Dashboard: GEOGLAM - https://earth.gov/data-catalog/geoglam https://github.com/NASA-IMPACT/veda-config-eic/blob/develop/datasets/geoglam.data.mdx?plain=1#L39 |
|
Thanks all for the great discussion, here are the updated designs: Link to Figma file 👉 https://www.figma.com/design/9INQauBWhiRxvOWDGhRrxO/US-GHG-Center?node-id=1781-3165&t=0jzLPDHhTvO3k5bR-1 |
There are several open tickets regarding color map functionality and UX:
Acceptance criteria
The text was updated successfully, but these errors were encountered: