Skip to content

Revert "Don't set $TPU_LIBRARY_PATH during import (#5698)"#5731

Merged
alanwaketan merged 1 commit intomasterfrom
revert-5698-wcromar/fix-tpu-path-env
Oct 25, 2023
Merged

Revert "Don't set $TPU_LIBRARY_PATH during import (#5698)"#5731
alanwaketan merged 1 commit intomasterfrom
revert-5698-wcromar/fix-tpu-path-env

Conversation

@alanwaketan
Copy link
Copy Markdown
Collaborator

This reverts commit 146f2a0.

It regresses the profiler.

Copy link
Copy Markdown
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.

hmm, I wonder why

@alanwaketan
Copy link
Copy Markdown
Collaborator Author

hmm, I wonder why

No ideas...

Copy link
Copy Markdown
Collaborator

@will-cromar will-cromar left a comment

Choose a reason for hiding this comment

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

Do you have any more details? If the profiler depends on $TPU_LIBRARY_PATH, we should fix this upstream.

@alanwaketan
Copy link
Copy Markdown
Collaborator Author

Do you have any more details? If the profiler depends on $TPU_LIBRARY_PATH, we should fix this upstream.

Do you have the more details you need in the gchat?

@alanwaketan alanwaketan merged commit 4baef3c into master Oct 25, 2023
jonb377 pushed a commit that referenced this pull request Oct 31, 2023
will-cromar added a commit that referenced this pull request Nov 13, 2023
will-cromar added a commit that referenced this pull request Nov 14, 2023
mbzomowski pushed a commit to mbzomowski-test-org/xla that referenced this pull request Nov 16, 2023
chunnienc pushed a commit to chunnienc/xla that referenced this pull request Dec 14, 2023
golechwierowicz pushed a commit that referenced this pull request Jan 12, 2024
bhavya01 pushed a commit that referenced this pull request Apr 22, 2024
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.

3 participants