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

✏️ Edit docs to use OGC:CRS84 lon/lat instead of EPSG:4326 #45

Merged
merged 2 commits into from
Aug 28, 2022

Conversation

weiji14
Copy link
Owner

@weiji14 weiji14 commented Aug 28, 2022

Auditing the coordinate reference system throughout the documentation and unit tests to use longitude/latitude (OGC:CRS84) instead of latitude/longitude (EPSG:4326) whenever suitable. I.e. an axis order of x/y instead of y/x.

Preview at https://zen3geo--45.org.readthedocs.build/en/45/vector-segmentation-masks.html#load-and-reproject-image-data

References:

Changing axis order from x/y instead of y/x! Decided to be a bit more precise after reading into OGC:CRS84 vs EPSG:4326, and audited the user-facing tutorials and API docs for good measure. Noticed too that the vector segmentation mask shapefile is actually in EPSG:9707 and not EPSG:4326, so fixed that. Also made minor edit on first walkthrough tutorial bumping torchdata links from 0.3.0 to 0.4.0.
Set the coordinate reference system for the geo-objects in test_datapipes_datashader.py to use OGC:CRS84 (lon/lat) instead of EPSG:4326 (lat/lon), just to be pedantic.
@weiji14 weiji14 added the documentation Improvements or additions to documentation label Aug 28, 2022
@weiji14 weiji14 added this to the 0.4.0 milestone Aug 28, 2022
@weiji14 weiji14 self-assigned this Aug 28, 2022
@weiji14 weiji14 merged commit 7f41032 into main Aug 28, 2022
@weiji14 weiji14 deleted the epsg_4326_to_ogc_crs84 branch August 28, 2022 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant