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

Fix zone letter functionality #12

Merged
merged 4 commits into from
Aug 1, 2016
Merged

Fix zone letter functionality #12

merged 4 commits into from
Aug 1, 2016

Conversation

psotres
Copy link
Contributor

@psotres psotres commented Mar 6, 2015

Fixes #11

@Turbo87
Copy link
Owner

Turbo87 commented Mar 6, 2015

you are correct that the latitude_to_zone_letter() function is a bit broken since it doesn't work for A, B, Y and Z (http://en.wikipedia.org/wiki/Universal_Transverse_Mercator_coordinate_system#Latitude_bands_2).

since the X zone ends at 84°N I'm not sure whether it is actually included or not. so you have any reference documentation that clear this up?

@psotres
Copy link
Contributor Author

psotres commented Mar 7, 2015

Here you can see some documentation: http://geokov.com/education/utm.aspx

"The northernmost band (X) is 12° latitude high instead of 8° and extends from 72°N to 84°N latitudes."

It does not explicitly mention that 84ºN is included on X band (or 80ºS included on C band). However, my understanding is that they are included because every lat/long point in the map has to have a UTM equivalent zone.

@psotres
Copy link
Contributor Author

psotres commented Mar 7, 2015

By the way, just to avoid opening another pull request, could you include on the __init__.py file both latlon_to_zone_number and latitude_to_zone_letter methods? They are very interesting for just evaluating if 2 different coordinates lies on the same UTM plane without needing to do the whole UTM conversion.

@Turbo87 Turbo87 merged commit 24b17a9 into Turbo87:master Aug 1, 2016
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

Successfully merging this pull request may close these issues.

Zone letter problem
2 participants