rsofun
rsofun copied to clipboard
Phydro merge into main
This is the discussion thread for planning the merge of the phydro
branch into the main model.
TODO before the merge:
- [ ] publish
rsofun
documentation paper as is
TODO for the merge:
- [x] fix #220
- [ ] fix #218
- [ ] calibration: reduce redundancies in cost-functions and streamline two methods of 'GenSA' or 'BayesianTools'
- [ ] update
rsofun::p_model_drivers
withreadRDS(file = here::here("data/p_model_drivers_newformat.rds"))
, as well asrsofun::p_model_drivers_vcmax25
- [ ] recompute
forcing
,forcing_24h
,forcing_daytime
,forcing_3hrmax
for the drivers
- [ ] recompute
- [ ] switch handling of
whc
from aparams_modl
to a site parameter- [ ] e.g. in
runread_pmodel_f.R
that sends the multi-sitedata.frame
to individual simualtions? - [ ] or alternatively remove it form par: here in R, here in C and here in F90.
- [ ] e.g. in
- [ ] include "regression tests" into the test suites to detect alteration of results, e.g. extend current test by verifying results remain similar
- [ ] design and document a robust, user-friendly interface for the different model flags
- [ ] ...
Other suggestions:
- [ ] reduce interface complexity:
- [ ] does
run_pmodel_f_bysite()
need to be an exported (and documented) function? Or will it anyway always be called fromrunread_pmodel_f
, and we can thus keep it internal (and undocumented)? - [ ] remove argument
parallel
form therunread_pmodel_f()
andrunread_biomee_f()
functions since it is equivalent with settingncores = 1
- [ ] ...
- [ ] does