Tanishq Jain
Tanishq Jain
@gaborznagy Hi, I am pretty new to open source but would love to work on this issue. Could you please help me with the issue more.
@infotroph I think once #3009 will be merged then only the CI check's for this PR will pass or we can revert the changes in `Rcheck.reference.log` for `utils` package and...
Hi @istfer, If we replace `as.numeric(mean(PEcAn.utils::ud_convert(diff(days_since_1700), "d", "s")))` with `as.numeric(mean(PEcAn.utils::ud_convert(as.numeric(diff(days_since_1700)), "d", "s")))` it works. I don't know if this is the right way to do it but the output I...
Hi @istfer have you updated the `met2CF.csv` and changed `as.numeric(mean(PEcAn.utils::ud_convert(diff(days_since_1700), "d", "s")))` to `mean(PEcAn.utils::ud_convert(as.numeric(diff(days_since_1700)), "d", "s"))` ?
@infotroph I think adding the above snippet is a great idea for now. Should I open a PR for it?
@infotroph I changed the function a little. Its working fine, but I am not sure it's the right method or not ``` ud_convert
@infotroph To change the `Rcheck_reference.log` PR #3000 is already opened. Should I merge it in this PR and close that one?
Hi @moki1202, I was searching for their description but wasn't able to find them. I was about to ask in the #cran channel regarding their description.
@infotroph @moki1202 Is something missing in `data.land`? the CI checks are failing because of it
@mdietze In case when no arguments passed for eg in line 105 of `modules/emulator/R/predict.GP.R` should we use ` utils::setTxtProgressBar()` directly? as we can not determine the min and max value