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.

Sign up for GitHub

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

Jump to bottom

Backport xla pin update (0612) #7261

Merged
merged 1 commit into from
Jun 13, 2024
Merged

Backport xla pin update (0612) #7261

merged 1 commit into from
Jun 13, 2024

Conversation

vanbasten23
Copy link
Collaborator

@vanbasten23 vanbasten23 commented Jun 12, 2024 •
edited
Loading

Backport #7254

@vanbasten23 vanbasten23 changed the base branch from master to r2.4 June 12, 2024 23:34
@bhavya01 bhavya01 self-requested a review June 12, 2024 23:37
bhavya01
bhavya01 approved these changes Jun 12, 2024
View reviewed changes
Copy link
Collaborator

@bhavya01 bhavya01 left a comment

Choose a reason for hiding this comment

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

Thanks!

@JackCaoG
Copy link
Collaborator

Can you also update the github issue

@JackCaoG
Copy link
Collaborator

also why is CI not running for r2.4 prs... this is weird

@vanbasten23 vanbasten23 merged commit da5843c into r2.4 Jun 13, 2024
@vanbasten23 vanbasten23 mentioned this pull request Jun 13, 2024
Open
@vanbasten23
Copy link
Collaborator Author

@bhavya01 @JackCaoG could you also approve #7262, the 2nd part of the pin update PR? That PR makes sure libtpu version matches the jax version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bhavya01 bhavya01 bhavya01 approved these changes

@will-cromar will-cromar Awaiting requested review from will-cromar

@JackCaoG JackCaoG Awaiting requested review from JackCaoG

@lsy323 lsy323 Awaiting requested review from lsy323

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@vanbasten23 @JackCaoG @bhavya01