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

ENH/MAINT: CNN regionalization, `smash.Net.add()` is deprecated #255

Closed nghi-truyen closed 1 month ago

nghi-truyen commented 2 months ago

Important notes:

nghi-truyen commented 2 months ago

@inoelloc Hello, Je viens de faire 2 PRs sur le changement de l'optim Fortran-Python et l'ajout de CNN (donc la méthode net.add a aussi changé). L'idée c'est de réduire au maximum les conflits sur ma branche dev (modèle hydride) et de faciliter les changments à faire pour chaque PR. Je propose de faire les PRs par cet ordre pour simplifier le boulot :

inoelloc commented 2 months ago

@inoelloc Hello, Je viens de faire 2 PRs sur le changement de l'optim Fortran-Python et l'ajout de CNN (donc la méthode net.add a aussi changé). L'idée c'est de réduire au maximum les conflits sur ma branche dev (modèle hydride) et de faciliter les changments à faire pour chaque PR. Je propose de faire les PRs par cet ordre pour simplifier le boulot :

* [Moving L-BFGS and SBS optimizers from Fortran to Python #238](https://github.com/DassHydro/smash/issues/238)

* [Add CNN mapping for regionalization #251](https://github.com/DassHydro/smash/issues/251)

* [New hydrid and ODE-based hydrological models #253](https://github.com/DassHydro/smash/issues/253)

* [Combine all optimizers from Fortran and Python #240](https://github.com/DassHydro/smash/issues/240)

* [mapping="ann" would be deprecated and replaced by "mlp" and "cnn" #254](https://github.com/DassHydro/smash/issues/254)

* [Saving the Model object and model ddt through the optimization process #252](https://github.com/DassHydro/smash/issues/252)

Hi, thanks. ok ok pour tout ca. J'aimerais finaliser les PR en cours avant de merge tout ca ^^"