Leap year issue for climate scenario

cgq-qgc / pyhelp

A Python library for the assessment of spatially distributed groundwater recharge and hydrological components with HELP

MIT License

17 stars 5 forks source link

Leap year issue for climate scenario #101

Open FHuchet opened 2 years ago

FHuchet commented 2 years ago

Bonjour Jean-Sébastien,

Un petit souci est apparu quand j'ai voulu lancer ma première simulation climatique avec les données futures :

Je pense que c'est lors de la vérification des années bissextiles dans le script (managers.py) :

Je préfère t'envoyer une "issue" que de modifier le script moi-même...

Merci pour ton retour, Bonne journée.

FHuchet commented 2 years ago

C'est lié au fait que nous travaillons avec seulement des années de 365 jours pour les simulations futures.

FHuchet commented 2 years ago

Voici l'erreur avec "raise Warning" :

FHuchet commented 2 years ago

Cela fonctionne en mettant des "#" pour éviter que le script fasse cette opération mais c'est sûr que ce n'est pas l'idéal...

FHuchet commented 2 years ago

Je vais mettre une autre issue pour la suite car j'ai une autre erreur ensuite avec le fichier de précipitations 9et sans doute les autres fichiers meteo).

jnsebgosselin commented 2 years ago

Comme discuté par courriel, HELP n'a pas une approche naïve pour la gestion des années bissextiles. C-à-d qu'une vérification est faite dans le code à partir du calendrier pour déterminer si une année est bissextile ou non et ceci est fait de façon indépendante aux données d'entrée.

Ainsi, même si on permettait les données météo sans année bissextile dans PyHELP, on ne serait pas plus avancé car HELP ne l'accepterait pas. Il faudrait donc apporter des modification au code source fortran de HELP pour que cela soit possible.