chirps icon indicating copy to clipboard operation
chirps copied to clipboard

Tests fail

Open adamhsparks opened this issue 4 years ago • 3 comments

I was troubleshooting some of the tests when I found discrepancies in the precip indices.

This is from https://github.com/ropensci/chirps/blob/7f60a8e6f9ddcf89f79179af4f5911a446e4a2ff/tests/testthat/test-get_chirps.R#L13

> x
 [1]  0.00  0.00  0.00  0.00 20.18 38.35  0.00 38.35  0.00  0.00
> chirps$chirps
 [1]  9.26  0.00  0.00  0.00 18.51 38.35  0.00 38.35  0.00  0.00

I didn't go further as it's getting late here. Just flagging this to follow up later.

adamhsparks avatar Aug 29 '21 13:08 adamhsparks

Thanks @adamhsparks I am in the middle of other tasks but I will take a look asap

K

kauedesousa avatar Aug 31 '21 08:08 kauedesousa

No worries. I didn't mean to hurry you. Just flagging this so we're aware.

adamhsparks avatar Aug 31 '21 08:08 adamhsparks

I've done quite a bit of work on the tests in the devel branch. I've isolated some of the issues with vcr cassettes thanks to @maelle. I'm adding more tests for some of the uncovered functionality as well.

adamhsparks avatar Sep 23 '21 02:09 adamhsparks