Skip to content
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

Bug in numexpr v2.8.5 - Fix in GeoIPS or pin to numexpr 2.8.4 until further notice #332

Closed
1 task
jsolbrig opened this issue Sep 18, 2023 · 0 comments · Fixed by #333
Closed
1 task

Bug in numexpr v2.8.5 - Fix in GeoIPS or pin to numexpr 2.8.4 until further notice #332

jsolbrig opened this issue Sep 18, 2023 · 0 comments · Fixed by #333

Comments

@jsolbrig
Copy link
Contributor

jsolbrig commented Sep 18, 2023

Requested Update

Description

There is a bug in numexpr 2.8.5 and above. They were attempting to fix some security issues involving the use of eval() and managed to break some behavior that previously worked. Our specific problem seems to relate to use of scientific notation in our equations.

For now, we will need to pin the numexpr version to 2.8.4 until further notice and track the changes in numexpr. For more information on the bug see pydata/numexpr#442.

The image below shoes the result of this problem.

image

Background and Motivation

  • [ OPTIONAL: Provide additional detailed information and background related to this issue ]

Alternative Solutions

  • [ OPTIONAL: If appropriate, list alternative fixes that could be considered]

Environment

  • [ IF APPROPRIATE: Corresponding repository versions ]

Code to demonstrate issue

  • [ IF APPROPRIATE: Can include steps here to identify/demonstrate bug fix or desired feature ]

Checklist for Completion

  • <!--- Include a list of steps that must be complete in order to close this Issue --->
@jsolbrig jsolbrig linked a pull request Sep 18, 2023 that will close this issue
8 tasks
@jsolbrig jsolbrig mentioned this issue Sep 18, 2023
8 tasks
@cpcamacho cpcamacho pinned this issue Sep 19, 2023
@jsolbrig jsolbrig unpinned this issue Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant