-
Notifications
You must be signed in to change notification settings - Fork 25
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
Projections and GeoFormatTypes #74
Comments
I think most of the projection code here can be removed once JuliaGeo/Proj.jl#51 lands. Would Proj4.jl not also be the place to add GeoFormatTypes.jl support, like was done in ArchGDAL? Would be nice if only a PROJ dependency would be sufficient for most of these conversions. Although GDAL supports a few more formats still I believe. |
Probably is! I just saw all the crs types defined here too. But better to support it at a lower level. Currently ArchGDAL does belessed type-piracy on |
Converting to GFT.ProjString should work here, would be a simple enough change. Will look into that after the larger refactor is complete :D |
It's worth noting that Proj.jl now depends on and accepts GeoFormatTypes since JuliaGeo/Proj.jl#74. |
#207 begins the path to adding this integration, as I recall. |
It should currently support EPSG, Proj-string, Proj-JSON, and WKT natively. I do still need to add tests though :D |
JuliaGeo has:
https://github.com/JuliaGeo/GeoFormatTypes.jl
It integrates with ArchGDAL for conversion between projection formats using
convert
and reprojection of points in any projection format usingArchGDAL.reproject
. It handles well known text, proj, epsg, gml and other projection formats without the handling package having to know what they are.It would be good if we could somehow integrate projections between these packages so that e.g. EPSG projections can be used here.
I'm looking to integrate this with GeoData.jl plotting at some stage. Currently I guess everything could be converted to
GeoFormatTypes.ProjString
usingconvert
and rewrapped asProjection
here? But it might be nice if any projection format just worked.The text was updated successfully, but these errors were encountered: