Ouranosinc / figanos / 22640588316

9%

Build Type

push

github

Committed by web-flow

Commit Message

Enregistrement des cmaps de l'IPCC dans matplotlib (#368)

<!-- Please ensure the PR fulfills the following requirements! -->
<!-- If this is your first PR, make sure to add your details to the
AUTHORS.rst! -->
### Pull Request Checklist:
- [ ] This PR addresses an already opened issue (for bug fixes /
features)
  - This PR fixes #xyz
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features).
- [ ] (If applicable) Tests have been added.
- [x] CHANGELOG.rst has been updated (with summary of main changes).
- [x] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added.

### What kind of change does this PR introduce?

* Suppression de `create_cmap`, on remplace par:
+ `get_ipcc_cmap_name` : Retourne le nom de la colormap à utiliser selon
les variables, la divergence ou le nom directement (auquel cas c'est pas
mal un pass-through)
+ `create_ipcc_cmap` : Renvoie un object colormap pour le nom IPCC de la
colormap. Fonction "privée", pas utile pour les usagers normaux.
* `create_ipcc_cmap` est appelé pour toutes les colormaps connues (i.e.
fichiers txt dans le dossier) et les objets créés sont enregistrés
directement dans `matplotlib.colormaps` et donc sont accessibles via
leur nom.

### Does this PR introduce a breaking change?
Oui. `utils.create_cmap` n'est plus. Il n'est pas nécessaire de créer
les colormaps à chaque fois.

### Other information:
J'ai comparé le temps d'importation de `figanos.matplotlib`
- sur `main`: ~3.9s
- sur cette branche : 4 s

Donc il semble que l'ajout ait un impact, mais 0.1 / 4 me semblait
acceptable.

Run Details

12 of 36 new or added lines in 2 files covered. (33.33%)

1 existing line in 1 file now uncovered.

173 of 1899 relevant lines covered (9.11%)

0.64 hits per line