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

Wrong IPCC colors #239

Open
aulemahal opened this issue Aug 27, 2024 · 3 comments
Open

Wrong IPCC colors #239

aulemahal opened this issue Aug 27, 2024 · 3 comments

Comments

@aulemahal
Copy link
Contributor

Generic Issue

  • figanos version: latest
  • Python version: all
  • Operating System: all

Description

The entries in the xlsx and txt files in https:/IPCC-WG1/colormaps and the entries in the txt file that we have here are different from the hex code in the json file.

For example "ssp370", which is the 4th "cat 2" ssp has 231, 29, 37 - > "#E71D25" in the three sources above, is set to "#DF0000" in the JSON.

Both color are quite similar, it is just weird that it's not exactly the same.

@juliettelavoie
Copy link
Contributor

bizarre bizarre! Je ne suis pas sur de l'origine du JSON. Il a été introduit dans cette PR (#22).

Notez aussi que selon ma recherche rapide les fichiers dans categorical_colors_rgb_0-255 ne sont en fait jamais utilisés.

@juliettelavoie
Copy link
Contributor

bon j'ai trouvé cette issue: #30
J'assume donc que le JSON est un création d'Alexis. Je ne sais pas comment/pourquoi il a fait un mapping entre RGB et hex. @sarahclaude tu te souviens ?

Sinon, je pense que ça vaudrait la peine de refaire ce mapping correctement.

@sarahclaude
Copy link
Collaborator

Non je ne m'en rappelle pas...
Je suis en train de regarder les RBG et Hex pour hvplot justement.
Si ce n'est pas urgent d'avoir le bon mapping, je pourrais le refaire en même temps que dans la création de cmap pour hvplot.

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

No branches or pull requests

3 participants