search

slgentil / croco

Code CROCO de base + CONFIGS qui contient les différentes configurations
0 stars 1 forks source link

minor error and suggestions for postp _adjust_grid #9

Closed NoeLahaye closed 4 years ago

NoeLahaye commented 4 years ago

I think l. 321 should read "_f0, _beta = None, None" istead of "f0, beta = None, none"

Puisque f est présent dans le netCDF grid, pourquoi ne pas l'utiliser comme coordonnée ? Ca évite d'avoir à donner f0, beta s'ils ne sont pas dans les fichiers de sortie (ce ne sont que des attributs) et de faire potentiellement des erreurs. Par contre on perdrai f_u, f_v, mais ils se recalculent très facilement avec grid.interp

Si OK je ferai les modifs nécessaires et ferai un pull request

apatlpo commented 4 years ago

oui et oui je dirais

pour le deuxième point je n'avais pas réalisé que f était aussi dans le fichier grille

@slgentil qu'en penses-tu

apatlpo commented 4 years ago

issue resolved by 4662a273fd69634bbc59861483f6d8e8d2baabaa

thanks @NoeLahaye !