Skip to content

Upgrade libtpu version to June to #3899

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

Merged
merged 1 commit into from
Aug 18, 2022
Merged

Upgrade libtpu version to June to #3899

merged 1 commit into from
Aug 18, 2022

Conversation

miladm
Copy link
Collaborator

@miladm miladm commented Aug 17, 2022

Upgrade libtpu version to June 23rd to align with that of our recent Tensorflow Pin #3794

@miladm miladm requested a review from yeounoh August 17, 2022 19:50
@miladm miladm self-assigned this Aug 17, 2022
@miladm miladm added the libtpu label Aug 17, 2022
@miladm miladm requested a review from JackCaoG August 17, 2022 19:53
@JackCaoG
Copy link
Collaborator

Did you have a chance to test it with the new tf?(You can just grab the wheel being build last night on TPUVM and test it with new libtpu)

@miladm
Copy link
Collaborator Author

miladm commented Aug 17, 2022

I am building it locally on the 06/23 image at the moment. Happy to try a more recent version too.

Copy link
Collaborator

@JackCaoG JackCaoG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will approve it for now, feel free to merge after you verify new libtpu works with new wheel.

@will-cromar
Copy link
Collaborator

Confirmed that this version of libtpu works. Can we merge this to fix our TPU tests?

@JackCaoG JackCaoG merged commit dd88cc5 into master Aug 18, 2022
@yeounoh yeounoh deleted the update_libtpu_pin_june branch January 7, 2023 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants