-
Notifications
You must be signed in to change notification settings - Fork 630
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
Pin numpy to <1.24 in TensorFlow examples #5534
Conversation
!build |
CI MESSAGE: [15934953]: BUILD STARTED |
!build |
CI MESSAGE: [15935241]: BUILD STARTED |
CI MESSAGE: [15934953]: BUILD PASSED |
CI MESSAGE: [15935241]: BUILD PASSED |
@@ -3,3 +3,4 @@ argparse-utils | |||
absl-py | |||
Pillow | |||
PyYAML | |||
numpy<1.24 |
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.
Why not <2
? Also, the description of the PR says "pin to 1.24" whereas it actually pins it to less than 1.24. I suggest renaming the PR (and keeping the code changes).
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.
Updated the commit message.
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.
Please rename the PR and adjust the commit message to reflect actual contents of the PR. Otherwise OK.
- due to a recent release of backward incompatible numpy 2.0 pin its version in the TensorFlow examples Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
pin its version in the TensorFlow examples
Category:
Other (e.g. Documentation, Tests, Configuration)
Description:
pin its version in the TensorFlow examples
Additional information:
Affected modules and functionalities:
Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A