Gr rainfall intensity

[asjeb]

Implementation of a GR model structure adapted to manage high rainfall intensity, inspired from Astagneau et al., called gr4_ri and gr5_ri.
For each new structure have been added 2 tunable parameters: one affecting infiltration into production reservoir and one affecting the partitioning of net rainfall between the two transfer branches.

[nghi-truyen]

Voilà ma review.

[nghi-truyen]

Pq pas mettre une citation pour Douinot et al. (2022) plutôt qu'un lien ?

[pag13]

La référence à citer est douinot et al. 2018 ; https://doi.org/10.5194/hess-22-5317-2018 (multi-hypothèses vs transferts latéraux, modele marine, bassins mediterranéens); je n'ai pas lu DOuinot 2022 (bassin et modele Luxembourg ?)

[pag13]

Implementation of an GR which allow to manage high rainfall intensity, inspired by P.C. Astagneau, call gr4_ri and gr5_ri with Adding of 2 optimized parameters. One on the instantaneous production and one on the partitionning coefficient of routing reservoir.

Rephrase please:
Implementation of a GR model structure adapted to manage high rainfall intensity, inspired from Astagneau et al., called gr4_ri and gr5_ri.
For each new structure have been added 2 tunable parameters: one affecting infiltration into production reservoir and one affecting the partitioning of net rainfall between the two transfer branches.

[pag13]

Well done!

Qu'en est il des aspects optimisation/bornes de ces nouveaux paramètres ? A discuter puis tester avec Ahmad, aussi avec imperviousness sur zone urbaine.

Je referai un passage sur la doc rst et les equations directement en sphinx, plus facile pour la lisibilité.

[pag13]

La référence à citer est douinot et al. 2018 ; https://doi.org/10.5194/hess-22-5317-2018 (multi-hypothèses vs transferts latéraux, modele marine, bassins mediterranéens); je n'ai pas lu DOuinot 2022 (bassin et modele Luxembourg ?)

[asjeb]

les valeurs par défaut :
3e-4, # % alpha1
1e-3, # % alpha2
les bornes :
(1e-6, 5e-4), # % alpha1 le range dans l'article est [0.01 - 0.1] * 10-3 s/mm, à 1e-6 ca correspond à 0.
(1e-5, 1.0), # % alpha2 -> [0.02 - 0.08] * 10-3 s/mm, 1 car proportion maximale de pluie qui couvre toute la gamme du coefficient de branche (0,1). alpha 1 a l'air de plus controler le processus. On voit des choses sur Qd jusqu' à du 3.10-1 sur la Cance. Le 1e-5 correspond à 0. à la 8e décimale sur le coefficient de branche.
J'ai annoncé à Ahmad que la branche avec en plus l'imperviouness était prête à l'emploi quand il en aurait besoin. L'imperviousness peut suivre sur main.

[pag13]

les valeurs par défaut : 3e-4, # % alpha1 1e-3, # % alpha2 les bornes : (1e-6, 5e-4), # % alpha1 le range dans l'article est [0.01 - 0.1] * 10-3 s/mm, à 1e-6 ca correspond à 0. (1e-5, 1.0), # % alpha2 -> [0.02 - 0.08] * 10-3 s/mm, 1 car proportion maximale de pluie qui couvre toute la gamme du coefficient de branche (0,1). alpha 1 a l'air de plus controler le processus. On voit des choses sur Qd jusqu' à du 3.10-1 sur la Cance. Le 1e-5 correspond à 0. à la 8e décimale sur le coefficient de branche. J'ai annoncé à Ahmad que la branche avec en plus l'imperviouness était prête à l'emploi quand il en aurait besoin. L'imperviousness peut suivre sur main.

Merci beaucoup Apolline.
As tu renseigné cela dans un commentaire dans le code ? par ex. près des bornes imposées.
Super que tu en aie informé Ahmad, il approfondira les tests, on pourra ensuite approfondir fonctionnalités et doc ;)

[asjeb]

Je rajoute les comments sur la branch test de Ahmad.