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

Use PyGMT instead of Cartopy in Chain tutorial #386

Merged
merged 2 commits into from
Oct 24, 2022
Merged

Use PyGMT instead of Cartopy in Chain tutorial #386

merged 2 commits into from
Oct 24, 2022

Conversation

leouieda
Copy link
Member

The PyGMT maps require less code and tend to look nicer than the Cartopy ones. It's also faster for non-rectangular maps. Add PyGMT and GMT as docs dependencies with pinned versions to avoid breaking the docs build. Replacing in a single tutorial to establish the infrastructure. Replace in other places in follow-ups.

Relevant issues/PRs:

The PyGMT maps require less code and tend to look nicer than the Cartopy
ones. It's also faster for non-rectangular maps. Add PyGMT and GMT as
docs dependencies with pinned versions to avoid breaking the docs build.
Replacing in a single tutorial to establish the infrastructure. Replace
in other places in follow-ups.

Co-authored-by: Santiago Soler <santisoler@fastmail.com>
@leouieda
Copy link
Member Author

@santisoler I took over your branch from 2020 and made it so we can do this one page at a time so that the PR doesn't live forever.

@leouieda leouieda marked this pull request as ready for review October 24, 2022 10:21
@leouieda
Copy link
Member Author

leouieda commented Oct 24, 2022

Aaand of course there are issues with Cartopy and GDAL. Have to pin matplotlib to avoid issues. (see SciTools/cartopy#2086)

@leouieda leouieda merged commit 9f271cf into main Oct 24, 2022
@leouieda leouieda deleted the pygmt branch October 24, 2022 12:58
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.

1 participant