phydro issues

[stineb]

@jaideep777

• [x] Resolve compilation error on M1 Mac (may test on a different platform with rhub checks, see below):

  dlopen(/Library/Frameworks/R.framework/Versions/4.4-arm64/Resources/library/00LOCK-rsofun/00new/rsofun/libs/rsofun.so, 0x0006): symbol not found in flat namespace '___gcc_nested_func_ptr_created'
Error: loading failed
Execution halted
ERROR: loading failed

I suspect this error arises because you have contains statements inside functions and further function definitions below that. Could you move all contains statements inside functions (only allowed at top level in module) and specify the functions outside?

• [x] Make sure that ET can be simulated with three options (a) SPLASH as done previously with AET considering diurnal cycle of net radiation and partial supply rate limitation (b) fully coupled to the atmosphere (T = 1.6 Gs D); (c) Penman-Monteith. Specify options through drivers$params_siml.
• [ ] Run devtools::check() and resolve all issues.
• [ ] Run rhub checks, see here.
• [x] vignettes folder must contain only .Rmd files. Scripts go into analysis/. Function definitions go into R/. Shell scripts go into src/
• [ ] Build website locally (pkgdown::build_site()) and clean it up - All .Rmd files in vignettes/ will appear as 'Articles'. This will be public-facing. Make it nice.
• [ ] Make sure that all FluxDataKit-related code is not in {rsofun}.
• [x] In all vignettes and scripts, do not read repo-external data. All data used by vignettes must be contained in the repo and scripts for data generation must be in data-raw/.
• [ ] Some functions defined in module md_photosynth_phydro are duplicated as they are already in module md_waterbal. Please remove them from md_photosynth_phydro.
• [ ] Could Penman-Montheith-specific code go into a md_waterbal_pml, see here? (maybe no so important)
• [ ] Avoid stop, print, and write statements in Fortran code (except for testing, but comment out upon commit).
• [x] Options (ET diffusion vs PM) to be set through drivers object (params_siml). I see it's in there but I see it's also set in md_photosynth_phydro.
• [x] Can you point me to how I use the Gs-P-model-coupled ET instead of SPLASH AET? Maybe in a separate vignette, that shows the two options like for GPP but for ET and also at FR-Pue?

I suspect that a clear separation between the model and the analysis done for your research @jaideep777 is needed. Keep only the very minimum in the {rsofun} repo such that you achieve a documentation of a very simplified example. Best is to use the data already contained in the repo (flux data for FR-Pue). All that goes beyond that and which is specific to what you did for the model calibration and evaluation on a larger set of sites should go in a separate (personal analysis) repository. Remember that our plan is to have {rsofun} on CRAN...