-
Notifications
You must be signed in to change notification settings - Fork 120
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
remove cartopy<0.20 requirement #733
Conversation
@knutfrode : do you remember why we had this in the first place? |
No, it seems like you added this 24 days ago, in relation to #708: ee4db8e#diff-9efd195f4e9bfb79ccd456a1d8370fafcc4bcb0b00ea3799222667d2ae818533 |
But we do get this now:
|
hm yes, seems like tests fail and we need to make some changes in plotting to work with new cartopy. |
Maybe we can get away with fixes like this:
|
No description provided.