Michael Dietze

Results 128 comments of Michael Dietze

Build is failing because of the following: ``` R check of modules/assim.sequential reports the following new problems. Please fix these and resubmit: checking dependencies in R code ... WARNING '::'...

@JoshuaPloshay just wanted to re-ping you on this before your semester gets busy

@sambhavnoobcoder as we discussed, the covariates come from the covariates.R file, which is in the open PR https://github.com/PecanProject/pecan/pull/3272 What I'd asked you to do wasn't to post a vague comment...

@juliusvira @istfer would love to get this PR in. It appears that @infotroph has submitted a PR to @juliusvira branch of pecan with some fixes that should resolve the GH...

Our apologies, that is a small helper function that's not exported because it wasn't intended for general use. The code comes from https://groups.google.com/forum/?fromgroups#!topic/ggplot2/-ZjRE2OL8lE and we borrowed it (with attribution) long...

@GandalfGwaihir @infotroph wanted to check in on the status of this PR and the requested changes. Is this something that can be wrapped up soon?

I don't have a stong objection, but I also file this under the category of "if it's not broke, don't fix it". There's a number of places in PEcAn where...

💯 agree this all needs to be documented and made much more clear, both for onboarding and as a checklist / standard operating procedure for the rest of us. FYI...

PR #874 adds download functions, but these still need to be integrated with met.process and the web code and tested

@infotroph it would be good to check if it is fully integrated with met.process and able to successfully work from the web page. I've never tried using it in practice,...