-
Notifications
You must be signed in to change notification settings - Fork 361
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
Add support for Equidistant Conic projection. #1022
Conversation
docs/make_projection.py
Outdated
@@ -42,7 +42,8 @@ | |||
PRJ_SORT_ORDER = {'PlateCarree': 1, | |||
'Mercator': 2, 'Mollweide': 2, 'Robinson': 2, | |||
'TransverseMercator': 2, 'LambertCylindrical': 2, | |||
'LambertConformal': 2, 'Stereographic': 2, 'Miller': 2, | |||
'LambertConformal': 2, 'EquidistantConic':2, |
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.
E231 missing whitespace after ':'
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.
Thanks @strezen, this is a really great addition 👍.
I've gone through the changes and am really happy with it.
lib/cartopy/crs.py
Outdated
false_easting=0.0, false_northing=0.0, | ||
standard_parallels=(20.0, 50.0), globe=None): | ||
""" | ||
Kwargs: |
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.
Not really your fault, but the docstring format has been updated since this branch was created.
See https://github.com/SciTools/cartopy/pull/998/files#diff-6dfcada343269e6678a777c4f9a5223d for reference.
@@ -0,0 +1,137 @@ | |||
# (C) British Crown Copyright 2015 - 2018, Met Office |
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.
Just 2018 will do 😄
@strezen I'm looking to try to clear out the last few things that were milestones 0.17. Can you rebase on current master and see if that fixes up the tests (at least Travis)? |
Implement support for Equidistant Conic along with tests from Snyder's book.
It needs the same central longitude wraparound fix as other projections.
Thanks @strezen! I took the liberty of rebasing this and fixing up things to match the current standard as well as support for Proj>=5. PTAL @dopplershift. |
Implement support for Equidistant Conic projection along with the related tests with data
taken from Snyder's book.