UMR-CNRM / EPyGrAM

Enhanced Python for Graphics and Analysis of Meteorological fields
Other
14 stars 15 forks source link

Geometry simplification #12

Closed SebastienRietteMTO closed 3 months ago

SebastienRietteMTO commented 1 year ago

Footprints removed from geometry objects (horizontal, vertical and spectral).

The structure argument of the horizontal geometries becomes a property and structure-specific geometry objects are suppressed.

The remaining Geometry file is split into several files.

AlexandreMary commented 4 months ago

Hello Seb, j'ai mergé cette branche mais lorsque je lance les tests j'ai 2 problèmes:

Des idées ? Alexandre

SebastienRietteMTO commented 3 months ago

La première erreur est due à pickle qui ne sait pas gérer les fonctions imbriquées. C'est corrigé dans le nouveau commit.

La seconde est due à la valeur par défaut (dans config.py) de netCDF_default_behaviour['H1D_is_H2D_unstructured']. En passant cette valeur à False le test passe. Le fichier netCDF de test "dims_V2D" est reconnu (avec cette option) comme un D3 flattened (même avant la PR). Je ne sais pas si on peut modifier sans dommages collatéraux cette valeur par défaut? Ou alors il faut modifier le test pour passer un adhoc_behaviour?

AlexandreMary commented 3 months ago

OK je modifie le test plutôt. Et merci pour le fix pickle