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
Soft deps in setup #123
Soft deps in setup #123
Conversation
Provide runtime checks for 'soft; dependency on Iris 2.1.
@pp-mo This is a really nice solution. Nice one. |
iris_grib/_load_convert.py
Outdated
@@ -43,6 +43,7 @@ | |||
ReferenceTarget | |||
from iris.util import _is_circular | |||
|
|||
from . import _confirm_iris_mercator_support |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry this seems to be bugged.
I think it should really be from .. import
??
Always a bit confused by relative imports ! 😕
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it works though, but now it can't find grib_phenom_translation
. Weird...
Yikes, I created a circular import problem ! |
Move mercator version-check to its own submodule, to avoid import problems.
@kaedonkers Ping! |
@bjlittle Ping |
Seconded -- can someone who is not me take a look please ?!? |
Popped a little function in the setup.py to look for iris 2.1 because that would be best, but if it's not available then it will install using iris 2.0 (or later).
Also added the eccodes dependency. gribapi lives in python-eccodes, but I guess the (official?) package name is actually just eccodes.