Michael Dietze
Michael Dietze
Similarly, @helge22a you should look at this too to see if there's anything that needs to be pulled in to get the daily forecast SDA working
If I'm interpreting this correctly, you're trying to merge the different SDA branches, but have actually created a new, 3rd branch. By not merging one into another, it's impossible to...
You can't move dbfiles.R to remote, it's core DB code and used in a lot of places. I think remote is just going to have to depend on DB (at...
I think that there are two high-level options: 1) just dump the test as a whole, since no one runs it and we're trying to shift to using the Benchmark...
From an install perspective, I think it'd be much simpler to have model2netcdf within the model container, not outside of it. Plus then the container only needs to return the...
@Hhh-hyc wanted to ping you about finishing up this PR, which has been in the queue for a while
``` R check of models/fates reports the following new problems. Please fix these and resubmit: checking dependencies in R code ... WARNING '::' or ':::' import not declared from: ‘ncdf4.helpers’...
Ok, so there's definitely an inconsistency here within model2netcdf.ED2 because on line 281 it has what appears to be the correct terms: ``` out
Did you recompile PEcAn so the package is built, and did you include the package in the dependencies for the ED package?
https://github.com/PecanProject/pecan/blob/master/models/ed/DESCRIPTION