DassHydro / smash

An open source, Python library interfacing the Fortran Spatially distributed Modelling and ASsimilation for Hydrology platform.
https://smash.recover.inrae.fr/
GNU General Public License v3.0
12 stars 6 forks source link

FIX: Beta parameter in GR percolation #158

Closed inoelloc closed 7 months ago

inoelloc commented 7 months ago
inoelloc commented 7 months ago

@nghi-truyen, il y a une erreur avec le test sur multiset estimate. J'ai du changer ce test parce qu'en modifiant la valeur de beta pour la percolation, le test renvoyait une erreur avec une division par zero de var_param quelque chose. En modifiant en local, j'avais plus les warnings et les tests passaient mais la sur la CI, ca ne passe pas. Est ce que tu aurais une idée pour modifier ces tests ?

nghi-truyen commented 7 months ago

@nghi-truyen, il y a une erreur avec le test sur multiset estimate. J'ai du changer ce test parce qu'en modifiant la valeur de beta pour la percolation, le test renvoyait une erreur avec une division par zero de var_param quelque chose. En modifiant en local, j'avais plus les warnings et les tests passaient mais la sur la CI, ca ne passe pas. Est ce que tu aurais une idée pour modifier ces tests ?

Ah oui, ça peut arriver, ça ne me surprend pas. C'est juste que la taille des samples est trop petite, donc cette erreur peut survenir. Je pense que la solution qui est moins coûteuse serait d'essayer de changer le random_state, sinon il faudra augmenter la taille des samples.

inoelloc commented 7 months ago

Finalement !

Yes :3. Pas tjs simple de changer pas grand chose

nghi-truyen commented 7 months ago

@inoelloc Aller je le merge pour pouvoir lancer les calculs avec le nouveau calcul de beta.