-
Notifications
You must be signed in to change notification settings - Fork 70
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
cartopy as an optional dependency #318
Comments
And perhaps the same for |
Hi @rafa-guedes, Glad you find Herbie useful. Yes, this is something I have thought about too, but never bothered to do it since I use Cartopy all the time. I wondered how long it would take before someone said something when I added scikit-learn as a new dependency; didn't take long 😆 Yes, I'd be open to a PR if you want to take a stab at it. Keeping it super simple with try/except statements should be good. Perhaps even matplotlib could be optional?
|
Thanks for providing this very useful library. I was wondering if you would consider making cartopy an optional dependency instead of a required one. Cartopy is a relatively large dependency which fails to build easily in some systems, and it is not required if one only requires the data access capability in the library. It looks like cartopy is used in the hrrr_zarr.py and accessors.py modules, perhaps some try/except conditional import in those modules with an appropriate exception message indicating the required dependency needs to be installed could work? I'm happy to submit a pull request if you think this is helpful.
The text was updated successfully, but these errors were encountered: