Chris Black
Chris Black
@its-Maniaco Please also add a Roxygen documentation string for the new `Rbinary` parameter by adding the followin between current lines 57 and 58 of `base/utils/R/convert.input.R`: ``` ##' @param Rbinary command...
@Its-Maniaco I took the liberty of working up a patch that implements the "read from host list" approach we recommended in the comments above. If you review and merge https://github.com/Its-Maniaco/pecan/pull/1...
* [x] Deprecate `mstmipvar` in favor of `standard_vars`? I could be convinced either way on this -- I know it's an eventual goal but there are still quite a few...
* [ ] Fix install-time warning `object ‘settings’ is created by more than one data call`
* [x] #1722 Move `do_conversions`, `get.results`, `run.write.configs` etc to a new package possibly named `PEcAn.workflow`. Rationale: These are the functions that integrate calls from many other packages, so by design...
* [x] Move Safelist to PEcAn.settings * [x] Move Safelist tests to match
* [ ] What to do with `base/utils/scripts/metutils.R`? * [ ] Delete `base/utils/scripts/time.constants.R`?
* [x] Review datasets #1713
* [x] Move `read.sa.output` and `write.sa.configs` to uncertainty
I think this definitely calls for changes inside `ud_convert` rather than in the code that calls it. This is a case that the udunits2 conversion did more or less handle,...