-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fixes for the pint
tests
#8983
base: main
Are you sure you want to change the base?
fixes for the pint
tests
#8983
Conversation
@@ -58,6 +58,8 @@ python -m pip install \ | |||
--upgrade \ | |||
--no-build-isolation \ | |||
git+https://github.com/pydata/bottleneck | |||
# manually install pint deps. These will be pulled in by the next released version, but for now we have to install them (since we use `--no-deps` pip will not pull them in the install). | |||
python -m pip install --upgrade 'flexcache>=0.3' 'flexparser>=0.3' 'appdirs>=1.4.4' typing_extensions |
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.
No big change but possibly this could just be python -m pip install --upgrade pint
, and that will do deps?
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.
that would work if the released version had the same dependencies as the upstream-dev version. Unfortunately, all of these (except typing_extensions
) have been added since the last release.
Another option would be to add python -m pip install --upgrade git+https://github.com/hgrecco/pint
(i.e. no --no-deps
for pint
)
This removes the use of the deprecated
numpy.core._exceptions.UFuncError
(and multiplication as a way to attach units), and makes sure we run thepint
tests in the upstream-dev CI again.