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

Upgrade to TF 2.7.0. #650

Merged
merged 6 commits into from Jan 4, 2022
Merged

Upgrade to TF 2.7.0. #650

merged 6 commits into from Jan 4, 2022

Conversation

MichaelBroughton
Copy link
Collaborator

@MichaelBroughton MichaelBroughton commented Jan 3, 2022

Supersedes #638

Also included:

  1. Reverted pip install instructions to correct release version of TensorFlow.
  2. Added python snippet to top of all test files to remove conflicting proto definitions on python path. I feel like there should be some way to do this in Bazel, but after trying for a few hours I'm out of ideas.

@zaqqwerty
Copy link
Contributor

Nice, looking forward to using TF 2.7 features with TFQ! Past TF version upgrades sometimes caused performance issues (see #568), how are things looking on that front?

@MichaelBroughton
Copy link
Collaborator Author

Not sure how performance looks. I think a good way to test will be to compare side-by-side some of the tutorial notebooks and see how much slower/faster the training loops get.

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.

None yet

2 participants