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

Improve documentation plotting of interrupted projections. #3144

Merged
merged 18 commits into from
Mar 28, 2022

Conversation

erykoff
Copy link
Contributor

@erykoff erykoff commented Mar 28, 2022

  • Tests added
  • Added clear title that can be used to generate release notes
  • Fully documented, including updating docs/source/*.rst for new API

This is a spin-off of #3139 that improves the documentation plotting code by adding checks for interrupted lines and graticules. This does not replace the continental outlines with filled continents because that is a harder job.

I also fixed a typo on the ob_tran plot so that it now renders as intended according to the description paragraph as an oblique Mollweide rather than oblique Miller Cylindrical, which is neither a common nor attractive projection.

Copy link
Member

@kbevers kbevers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, very nice to get cleaner versions of the problematic figures!!

@kbevers
Copy link
Member

kbevers commented Mar 28, 2022

@erykoff do you mind if I squash this into one commit? All the commits are working towards the same thing so I think it makes more sense to group them in one.

@erykoff
Copy link
Contributor Author

erykoff commented Mar 28, 2022

No problem. I left them all as separate commits in case you had some comments on particular ones. Otherwise, it's fine to bundle it up.

@kbevers kbevers merged commit 7d8e728 into OSGeo:master Mar 28, 2022
@erykoff erykoff deleted the doc_plotting branch March 28, 2022 14:45
github-actions bot pushed a commit that referenced this pull request Mar 28, 2022
Improves the documentation plotting code by adding checks for interrupted lines and graticules. This does not replace the continental outlines with filled continents because that is a harder job. A number of plots has been improved and included in the docs.

Also fixes a typo on the ob_tran plot so that it now renders as intended according to the description paragraph as an oblique Mollweide rather than oblique Miller Cylindrical, which is neither a common nor attractive projection.
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.

2 participants