Sitelet https://github.com/SHTOOLS/SHTOOLS/pull/448
Skip to content

Remove Cartopy build-time dependencies - #448

Merged
MarkWieczorek merged 2 commits into
SHTOOLS:developfrom
QuLogic:no-cartopy-deps
Mar 5, 2024
Merged

MarkWieczorek merged 2 commits into
SHTOOLS:developfrom
QuLogic:no-cartopy-deps

Conversation

@QuLogic

@QuLogic QuLogic commented Feb 27, 2024

Copy link
Copy Markdown
Member

Latest Cartopy produces wheels, so there's no need to install its build-time dependencies ourselves.

Reminders

  • Base all changes on the develop branch: the master branch is used only when releasing new versions.
  • Run make check to ensure that the python code follows standard formatting conventions.
  • [n/a] If adding new features, update the docstring to provide all information that is required to use the feature.

@MarkWieczorek

Copy link
Copy Markdown
Member

Does Cartopy need proj and geos? We should update the "how to install cartopy" section here: https://github.com/QuLogic/SHTOOLS/blob/no-cartopy-deps/docs/pages/mydoc/python-installing.md

If this is all moot, we can just delete the whole section.

@QuLogic

QuLogic commented Feb 28, 2024

Copy link
Copy Markdown
Member Author

That's all hidden behind pyproj, which has a pre-built wheel. As shown in the Cartopy link there, no proj or geos any more.

Latest Cartopy produces wheels, so there's no need to install its
build-time dependencies ourselves.
@MarkWieczorek
MarkWieczorek merged commit 3f396e4 into SHTOOLS:develop Mar 5, 2024
@MarkWieczorek

Copy link
Copy Markdown
Member

looks good. Thanks!

@QuLogic
QuLogic deleted the no-cartopy-deps branch March 5, 2024 23:29
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