Make cartopy an optional dependency - Githubissues

mantle-convection-constrained / terratools

Tools to read, analyse and visualise models written by the TERRA mantle convection code

https://terratools.readthedocs.io/en/latest/

MIT License

7 stars 5 forks source link

Make cartopy an optional dependency #85

Closed anowacki closed 1 year ago

anowacki commented 1 year ago

This means that users must install cartopy separately if they want to make maps which use cartopy, via pip or otherwise.

Update the test script to install cartopy unconditionally. A future commit will add a test for the correct behaviour (an ImportError) when trying to call a function which relies on cartopy.

Pull Request Checklist. Please read and check each box with an X. Delete any part not applicable.

Describe what you did in this PR and why you did it.

For all pull requests:

[x] I have run the contrib/utilities/indent script from the main directory to indent my code.

For new features/models or changes of existing features:

[x] I have tested my new feature locally to ensure it is correct.
[x] I have created a testcase for the new feature/benchmark in the tests/ directory.
[ ] I have added a changelog entry in the doc/changes directory that will inform other users of my change.