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

Leap year issue for climate scenario #101

Open
FHuchet opened this issue May 9, 2022 · 5 comments
Open

Leap year issue for climate scenario #101

FHuchet opened this issue May 9, 2022 · 5 comments

Comments

@FHuchet
Copy link
Member

FHuchet commented May 9, 2022

Bonjour Jean-Sébastien,

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

image

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

image

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

Merci pour ton retour,
Bonne journée.

@FHuchet
Copy link
Member Author

FHuchet commented May 9, 2022

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

@FHuchet
Copy link
Member Author

FHuchet commented May 9, 2022

Voici l'erreur avec "raise Warning" :
image

@FHuchet
Copy link
Member Author

FHuchet commented May 9, 2022

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...

image

@FHuchet
Copy link
Member Author

FHuchet commented May 9, 2022

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 jnsebgosselin changed the title Leap year issue for climate scenarii Leap year issue for climate scenario May 12, 2022
@jnsebgosselin
Copy link
Member

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.

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

2 participants