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

Round coordinates to 6 digits #28

Open
AntoineAugusti opened this issue Oct 31, 2018 · 4 comments
Open

Round coordinates to 6 digits #28

AntoineAugusti opened this issue Oct 31, 2018 · 4 comments

Comments

@AntoineAugusti
Copy link

Hello!

Thanks for this repository. It would be an improvement to round all coordinates to 6 digits to have smaller files and to avoid unnecessary digits:

Accuracy of latitude / longitude:

The sixth decimal place is worth up to 0.11 m: you can use this for laying out structures in detail, for designing landscapes, building roads. It should be more than good enough for tracking movements of glaciers and rivers. This can be achieved by taking painstaking measures with GPS, such as differentially corrected GPS.

From https://gis.stackexchange.com/questions/8650/measuring-accuracy-of-latitude-and-longitude

@gregoiredavid
Copy link
Owner

Thanks for the suggestion ! All coordinates should already be rounded to 6 digits since version 2.0.2 (https://github.com/gregoiredavid/france-geojson/releases/tag/v2.0.2)

Did you find a file where they are not ? I may have forgotten some.

@AntoineAugusti
Copy link
Author

@AntoineAugusti
Copy link
Author

After a search with the regex \.[0-9]{6,}, I get this list:

  • arrondissements-version-simplifiee.geojson
  • cantons-avec-outre-mer.geojson
  • cantons-version-simplifiee.geojson
  • cantons.geojson
  • communes-version-simplifiee.geojson
  • departements-version-simplifiee.geojson
  • departements/35-ille-et-vilaine/cantons-35-ille-et-vilaine.geojson
  • departements/50-manche/cantons-50-manche.geojson
  • metropole-version-simplifiee.geojson
  • regions-avant-redecoupage-2015.geojson
  • regions-version-simplifiee.geojson
  • regions/bretagne/cantons-bretagne.geojson
  • regions/normandie/cantons-normandie.geojson

@gregoiredavid
Copy link
Owner

Thank you very much for the list ! I will look into it and fix this ASAP.

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

No branches or pull requests

2 participants