-
Notifications
You must be signed in to change notification settings - Fork 644
Add support for TensorFlow 2.17 #974
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
Conversation
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 had one question about Keras in the requirements, but otherwise, LGTM.
I just realized that this uses a slightly different way of generating requirements.txt. I will see if I can get MBB to review #945 today; in the meantime, can you try to generate requirements.txt using the script generate_requirements.sh in that PR? I think it will be merged soon, and so if this PR for 2.17 is merged afterwards, it will update the requirements using the same procedure, and nothing more will need to be done. (Conversely, if this is merged before the other PR, then we might have to regenerate the requirements using that script and do another PR just to update requirements.txt a 3rd time.)
|
One last tiny request: could you make the title slightly more descriptive? E.g., spell out TensorFlow and say it's about supporting version 2.17. I don't mean to be picky – it's just to help make the commit history be clearer. |
mhucka
left a comment
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.
It's great to see this. Thanks!
Summary
requirements.inand usepip-compileto producerequirements.txtKey changes
@org_tensorflowto 2.17.1requirements.in:requirements.txtis produced by runningpip-compile --upgrade --generate-hashes -o requirements.txt requirements.inTesting
bazel build ... release:build_pip_packagesucceedsscripts/test_all.sh&./scripts/ci_validate_tutorials.sh) passNotes