-
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
adds ability to export non-EPSG CRSs URIs #93
adds ability to export non-EPSG CRSs URIs #93
Conversation
…d code from the CRS directly
okay so I know that the None object is most likely due to the too-high confidence level default to the to_authority call, although I can't seem to find much information on why proj doesn't always have high confidence, is there information missing from the CRS of that polar projection? |
okay so although tests (mostly) pass, I am going to add back a check to ensure we don't try to unpack a None to allow for cases when the right answer really is EPSG/0/None |
…he authority don't raise an exception
@vincentsarago I've added a test (one parameterized test per authority, not checking every code) and fixed remaining black formatting issues. I could see adding a test to try to ensure that EPSG/0/None is at least returned sometimes, your call. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
grabs the authority, version and code from the CRS directly to encode the URIs. Related to #92 but can be merged separately. I'm not aware of any issues this would cause but I am open to hearing about them