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

Load/save CRS information #87

Closed
13 tasks done
juliohm opened this issue Jun 6, 2024 · 8 comments
Closed
13 tasks done

Load/save CRS information #87

juliohm opened this issue Jun 6, 2024 · 8 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@juliohm
Copy link
Member

juliohm commented Jun 6, 2024

Now that the stack supports CRS, we need to account for that in some file formats.

Read

  • GeoJSON
  • Shapefile
  • GeoPackage
  • GeoParquet
  • GeoTIFF
  • NetCDF
  • GRIB

Write

  • GeoJSON
  • Shapefile
  • GeoPackage
  • GeoParquet
  • GeoTIFF
  • NetCDF
@juliohm juliohm added enhancement New feature or request help wanted Extra attention is needed labels Jun 6, 2024
@juliohm juliohm changed the title Read/write CRS information from file Read/write CRS information Aug 2, 2024
@juliohm juliohm changed the title Read/write CRS information Load/save CRS information Aug 14, 2024
@juliohm
Copy link
Member Author

juliohm commented Dec 27, 2024

Now that JuliaGeo/GeoParquet.jl#25 is merged, we can address this issue.

@juliohm
Copy link
Member Author

juliohm commented Jan 5, 2025

@eliascarv do we have all the ingredients in place to complete this issue?

@eliascarv
Copy link
Member

Yes, now you just need to get the EPSG code from ProjJSON.

@juliohm
Copy link
Member Author

juliohm commented Jan 10, 2025

Apparently our asgeotable function already extracts the CRS from the table:

crs = GI.crs(table)

I tried with a couple of sample files, and they still load the CRS as Cartesian{NoDatum}.

@eliascarv
Copy link
Member

eliascarv commented Jan 10, 2025

Now, the only thing left is to parse the GFT.GeoJSON object in the crstype function.

@juliohm
Copy link
Member Author

juliohm commented Jan 13, 2025

Thanks for adding the missing projjson in #146 . I didn't check the save part of this issue yet, but please feel free to close it if it is already addressed.

@eliascarv
Copy link
Member

PR #146 only adds support for loading ProjJSON.

@juliohm
Copy link
Member Author

juliohm commented Jan 14, 2025

Fixed on the master branch.

@juliohm juliohm closed this as completed Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants