courtiol / IsoriX

This is the GitHub repository dedicated to the development of the R package IsoriX
13 stars 6 forks source link

Updates to elevatr: Moving to sf and terra #167

Closed jhollist closed 1 year ago

jhollist commented 1 year ago

First, thanks for using elevatr in your package on CRAN. Really excited that others have found it useful.

This issue is to inform you that there are some big changes coming up shortly with elevatr. There are all around the archiving of rgdal and rgeos and the associated changes to sp and raster. I have decided to move entirely to sf and terra and am removing dependencies to rgdal, rgeos, sp, and raster. This has been implemented in the latest version on the main branch on GitHub (https://github.com/jhollist/elevatr). I have not completely removed the raster dependency and elevatr is still returning a RasterLayer from get_elev_raster. This will change in a future release and elevatr will return only sf objects for point elevation and terra SpatRasters for the elevation rasters. Given this you will need to account for this change in your packages.

I am planning on sending version 1.0, which still returns RasterLayer, to CRAN in a few days. At that point I will update the GitHub version to return SpatRasters which should allow you all to test your packages. I would expect to wait several months before sending the SpatRaster version to CRAN.

courtiol commented 1 year ago

My package no longer relies on raster or sp (see branch terra under devel).