Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compatibility gdal3 proj6 #384

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

dschlaep
Copy link
Member

@dschlaep dschlaep commented Oct 6, 2020

  • Minimal updates to accommodate GDAL3+/PROJ6+
  • Introduces new dependency on package rSW2st

- move function `crs_units` to new package `rSW2space` to handle different crs
- make sure that we have a sufficiently new `rgdal` for the "new generation" crs argument checks
only white-space changes
(Unpublished) package `rSW2space` renamed to `rSW2st` (spatio-temporal)
make it easier to install our set of package dependencies
only white-space adjustments; no changes to the code
- use `sf` for handling pre- and post- GDAL3+/PROJ6+ correctly (see https://www.r-spatial.org/r/2020/03/17/wkt.html)

- use `sf::st_crs()` for all coordinate reference system functionality:
* changed (almost all instanced of) `sp::CRS()` and `raster::crs()` to `as(sf::st_crs(), "CRS")`
* changed `raster::compareCRS()` to `sf::st_crs() == sf::st_crs()`
* replace all crs encoded as proj4 character strings with epsg codes

- crs of daymet which is a Lambert Conformal Conic projection is still a proj4 representation;
--> todo: convert to WKT2

--> todo: replace all `sp` with `sf` functionality - where possible
- moved `methods` to "Depends"
- moved `rSOILWAT2`, `rSW2utils`, `rSW2st`, and `rSW2fun` to "Imports"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant