search

r-spatial / dtwSat

Time-Weighted Dynamic Time Warping for satellite image time series analysis
https://www.victor-maus.com/dtwSat/
GNU General Public License v3.0
131 stars 39 forks source link

Error in `split.stars(x, c("time"))`: length(x) == 1 is not TRUE #62

Closed vwmaus closed 1 year ago

vwmaus commented 1 year ago

Error in Flavors r-oldrel-macos-arm64 and r-oldrel-macos-x86_64.

Details in https://cran.r-project.org/web/checks/check_results_dtwSat.html

vwmaus commented 1 year ago

@edzer, CRAN results show the following error on the r-oldrel-macos-arm64 and r-oldrel-macos-x86_64 flavors. Could you point me in the right direction for addressing this? Or is it an issue that needs to be resolved within stars itself?

     Error in `split.stars(x, c("time"))`: length(x) == 1 is not TRUE
     Backtrace:
     ▆
     1. ├─base::system.time(...) at test-twdtw_classify.R:24:0
     2. └─dtwSat::twdtw_knn1(...)
     3. ├─base::split(x, c("time"))
     4. └─stars:::split.stars(x, c("time"))
     5. └─base::stopifnot(length(x) == 1)
edzer commented 1 year ago

Thanks! Those two platforms have a broken GDAL install (no netcdf linked to GDAL), so stars tests break on them, and old stars versions are kept/used. This will go away when a new release of R appears, and "oldrel" moves up to current release - no need to take action.

edzer commented 1 year ago

see https://github.com/r-spatial/stars/issues/566

vwmaus commented 1 year ago

Thanks!