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

[build] Specify numpy < 2.0 in build.requirements.txt #3078

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tdietert
Copy link
Contributor

When running the e2e tests, specifically python -m e2e_testing.main, many tests fail when numpy-2.X.X is installed, and an error message stating numpy-1.X.X is required. We should reflect this in the build.requirements.txt.

When running the e2e tests, specifically `python -m e2e_testing.main`,
many tests fail when numpy-2.X.X is installed, and an error message
stating numpy-1.X.X is required. We should reflect this in the
build.requirements.txt.
@tdietert
Copy link
Contributor Author

tdietert commented Mar 29, 2024

Should this be a requirement in the test.requirements.txt instead? Running python -m pip install -r requirements.txt installs all the packages in all the XXX.requirements.txt files, so I don't think it matters.

@stellaraccident
Copy link
Collaborator

I've been watching these issues make their rounds on some different projects and each seems to be deciding it differently. I'd like to understand better what is going on.

@vivekkhandelwal1
Copy link
Collaborator

Hi @tdietert, I don't see any failures with the current requirements. Can you please elaborate the issue in more detail? If you're not encountering the issue anymore, then please close this PR.

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 this pull request may close these issues.

3 participants