search

BaRatin-tools / BaRatinAGE

BaRatin Advanced Graphical Environment
GNU General Public License v3.0
4 stars 0 forks source link

add min and max stage values to the rating curve equation #1

Open JeromeLeCoz opened 1 year ago

JeromeLeCoz commented 1 year ago

1/ remplacer les ≤ par des < pour éviter les pbs d'encodage.

2/ indiquer les hmin/hmax de la grille de calcul de la CT comme valeurs extrêmes d'application du modèle

IvanHeriver commented 1 year ago

Here is an example of rating curve export illustrating the issue 1/:

h = -5.26347 : Q = 0
-5.26347 < h = 1.44294 : Q = 45.1791*(h+5.26347)^1.65066
1.44294 < h = 6.99758 : Q = 126.807*(h+1.98058)^1.71393
6.99758 < h = 8.45361 : Q = 155.211*(h+1.00658)^1.71138
h > 8.45361 : Q = 155.211*(h+1.00658)^1.71138 + 82.3314*(h-8.45361)^1.66879

where there's only = instead of ≤ in the ranges at the start of the lines.

  1. Ok to replace ≤ with <= (not < only right?). Or solve the encoding issue.
  2. minimal height is already clear I think, a maximum height could be added using the hmax defined by the user in BaRatinAGE. This falls in the "enhancement" category and will moleky be taken care of only for v3 of BaRatinAGE.
JeromeLeCoz commented 1 year ago
  1. Pas fan de "<=" qui peut ne pas être compris, pris pour une flèche, etc. Je préfère "<" qui n'est pas faux mathématiquement ici...
  2. hmin n'est pas égale au premier h (=b1) du modèle, [hmin;hmax] est l'étendue de la grille sur laquelle on s'autorise à calculer la CT. Je pense que si on décide de fournir les paramètres de la grille, il faut les séparer clairement des paramètres de l'équation de la CT, par exemple en affichant plus loin en bas de fichier :

Grid parameters: h_min = ... h_max = ... Nstep = ... Hstep = ...

IvanHeriver commented 1 year ago

Exported equations now use strict symbols < instead ≤.

I remove the bug tag. The requested feature regarding the stage values will be addressed in futur version of BaRatinAGE.