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

[ROCm] Fix run single gpu script to avoid duplicates #22924

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

Ruturaj4
Copy link
Collaborator

@Ruturaj4 Ruturaj4 commented Aug 8, 2024

@superbobry can you please review and merge? thanks!

The script will help improve our internal reporting.

@Ruturaj4 Ruturaj4 force-pushed the rocm-jax-stable-2024_08_07 branch 2 times, most recently from 11bb8f7 to 22be05b Compare August 8, 2024 04:46
build/rocm/run_single_gpu.py Outdated Show resolved Hide resolved
build/rocm/run_single_gpu.py Outdated Show resolved Hide resolved
@google-ml-butler google-ml-butler bot added kokoro:force-run pull ready Ready for copybara import and testing labels Aug 8, 2024
@Ruturaj4 Ruturaj4 closed this Aug 9, 2024
@Ruturaj4 Ruturaj4 deleted the rocm-jax-stable-2024_08_07 branch August 9, 2024 14:55
@Ruturaj4 Ruturaj4 restored the rocm-jax-stable-2024_08_07 branch August 10, 2024 01:17
@Ruturaj4
Copy link
Collaborator Author

@superbobry is this not merged yet? just closed? Could you please check?

@dfm
Copy link
Collaborator

dfm commented Aug 16, 2024

@Ruturaj4 — It looks like you closed this PR. Did you not mean to?

@Ruturaj4 Ruturaj4 reopened this Aug 16, 2024
@Ruturaj4
Copy link
Collaborator Author

@dfm Just reopened again, idk how that was closed!

@dfm
Copy link
Collaborator

dfm commented Aug 16, 2024

Thanks - can you rebase onto main? It looks like there are conflicts now.

@Ruturaj4
Copy link
Collaborator Author

Thanks - can you rebase onto main? It looks like there are conflicts now.

Done, thanks!

@dfm
Copy link
Collaborator

dfm commented Aug 19, 2024

@Ruturaj4 — Thanks! Please also squash the commits into one and we'll get this merged.

@Ruturaj4
Copy link
Collaborator Author

@dfm thanks! I will keep in mind two squash commits in future. can you please approve?

@dfm dfm self-assigned this Aug 20, 2024
@copybara-service copybara-service bot merged commit 7758a9c into jax-ml:main Aug 20, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull ready Ready for copybara import and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants