-
Notifications
You must be signed in to change notification settings - Fork 0
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
Home for CRS objects? #1
Comments
@paleolimbot I'm not sure that could work out, and whether you'd want that - |
Got it! I'll keep this experimental. |
How about a standalone package maybe "crsdef"? That contains the CRS definition? This package would be more like an "engine" (sf would also be an engine). |
Would that package only contain the crs definitions, or also do the transformations & conversions? One other thing to take into account is axis order, see e.g. https://www.r-spatial.org/r/2020/03/17/wkt.html#axis-order , but |
Just definitions, I think. Without a copy of PROJ or GDAL it wouldn't be able to completely validate, but it could probably at least validate the type/format. From the PR you referenced earlier it sounds like the new CRS object is just storing the user input and auto-generating the other info on the fly. Off the top of my head, the functions I'm happy to write that up as a package if it seems like it's useful! |
Where are these functions hosted? PROJ is software around a central sqlite3 database, now in |
I did some experimenting...I think quite a bit as possible without installing PROJ. You can cache the (not offended if this is foolhardy or makes your implementation difficult, but I think that a zero-dependency package that just does CRSes is possible and might be nice if it can be done well) |
Nice!! I see nothing against this if this is what users want / if this solves problems (whether yours or others'); do also look at the PROJ and proj4 packages on CRAN. I do see the potential issue of yet another stand-alone software+database package that can run (or be) out of sync with other R packages expecting to be in sync, or other non-R parts of your system (PostGIS, QGIS, ...) people work with. |
I envision these objects as communicating intent...resolving proj4 and WKT + partial validation and pretty error messages would mostly be about convincing maintainers to use them. In all cases, the input would be interpreted by the sfs, rasters, rgdals, and sps of the R universe. |
@edzer - Any chance I can convince you to house CRS objects here? I haven't quite gotten this to pass the CMD check (complains about .hpp files and missing sqlite3), but it compiles on my machine without errors or warnings. It has an internal copy of PROJ7, which might allow all platforms to access the latest and shiniest PROJ. I'd prefer to use your CRS object than make a new one with converters!
The text was updated successfully, but these errors were encountered: