Please remove dependencies on **rgdal**, **rgeos**, and/or **maptools**

[rsbivand]

This package depends on (depends, imports or suggests) raster and one or more of the retiring packages rgdal, rgeos or maptools (https://r-spatial.org/r/2022/04/12/evolution.html). Since raster 3.6.3, all use of external FOSS library functionality has been transferred to terra, making the retiring packages very likely redundant. It would help greatly if you could remove dependencies on the retiring packages as soon as possible.

[rsbivand]

You will be aware, for example from:
https://r-spatial.org/r/2022/04/12/evolution.html,
https://r-spatial.org/r/2022/12/14/evolution2.html,
https://r-spatial.org/r/2023/04/10/evolution3.html and
https://rsbivand.github.io/csds_jan23/bivand_csds_ssg_230117.pdf and
perhaps view https://www.youtube.com/watch?v=TlpjIqTPMCA&list=PLzREt6r1NenmWEidssmLm-VO_YmAh4pq9&index=1
that rgdal, rgeos and maptools will be retired this
year, in October 2023.

The package imports 'rgdal', specifically rgdal::writeOGR in NAMESPACE and line 37 in export.R. Ad discussed in https://r-spatial.org/r/2023/04/10/evolution3.html and in links given there, and since you use raster which depends on terra, coercion to vect and terra::writeVector probably provide the easiest mitigation. Please act very soon, best by June, latest by October 2023.

[achubaty]

Thank you, I will make the changes in the next few weeks.

[achubaty]

removed dependencies rgdal and rgeos in e10b0ab.

converting from sp/raster to sf/terra will take much more work (#69)

[ecologics]

Thanks for doing that. Do we have to do the shift to terra or is that just preferred? Paul

…

________________________________ From: Alex Chubaty ***@***.***> Sent: Tuesday, April 18, 2023 9:46:38 PM To: achubaty/grainscape ***@***.***> Cc: Subscribed ***@***.***> Subject: Re: [achubaty/grainscape] Please remove dependencies on **rgdal**, **rgeos**, and/or **maptools** (Issue #68) [△EXTERNAL] removed dependencies rgdal and rgeos in e10b0ab<e10b0ab>. converting from sp/raster to sf/terra will take much more work (#69<#69>) — Reply to this email directly, view it on GitHub<#68 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AEIDEMEQU2IVR2WTIK2XYJLXB5NZ5ANCNFSM6AAAAAAS6G7S4U>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[achubaty]

We should definitely move to using it (and still allow RasterLayer objects to be used). It's straightforward to do so, but will require converting most of the package code and adding corresponding tests.

[rsbivand]

Yes, it is very sensible adding tests so that you can see whether the changes in the underlying software impact output (this both includes dropping retiring packages, and the frequent updates in PROJ, GDAL and GEOS).