Skip to content

Conversation

@jataylo
Copy link
Collaborator

@jataylo jataylo commented Nov 4, 2024

#136792 accidentally disabled flex attention UTs on ROCm. Re-enabling.

cc @jeffdaily @sunway513 @jithunnair-amd @pruthvistony @ROCmSupport @dllehr-amd @hongxiayang @naromero77amd

@jataylo jataylo added rocm This tag is for PRs from ROCm team rocm priority high priority ROCm PRs from performance or other aspects ciflow/rocm Trigger "default" config CI on ROCm ciflow/inductor-rocm Trigger "inductor" config CI on ROCm labels Nov 4, 2024
@jataylo jataylo requested a review from mruberry as a code owner November 4, 2024 16:40
@pytorch-bot
Copy link

pytorch-bot bot commented Nov 4, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/139632

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 1f15726 with merge base 419a7e1 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot bot added the module: rocm AMD GPU support for Pytorch label Nov 4, 2024
Copy link
Contributor

@drisspg drisspg left a comment

Choose a reason for hiding this comment

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

Thanks!

@drisspg drisspg added the release notes: rocm mandatorylabel label Nov 4, 2024
@jataylo jataylo added the keep-going Don't stop on first failure, keep running tests until the end label Nov 4, 2024
@jataylo
Copy link
Collaborator Author

jataylo commented Nov 4, 2024

Rebasing to run CI again but with keep-going label.

@jataylo
Copy link
Collaborator Author

jataylo commented Nov 4, 2024

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Tried to rebase and push PR #139632, but it was already up to date. Try rebasing against main by issuing:
@pytorchbot rebase -b main

@jataylo
Copy link
Collaborator Author

jataylo commented Nov 5, 2024

Opened issue for failure #139754 will skip UT for now and merge this.

@jataylo
Copy link
Collaborator Author

jataylo commented Nov 6, 2024

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 6, 2024
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

pobin6 pushed a commit to pobin6/pytorch that referenced this pull request Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/inductor-rocm Trigger "inductor" config CI on ROCm ciflow/rocm Trigger "default" config CI on ROCm ciflow/trunk Trigger trunk jobs on your pull request keep-going Don't stop on first failure, keep running tests until the end Merged module: rocm AMD GPU support for Pytorch open source release notes: rocm mandatorylabel rocm priority high priority ROCm PRs from performance or other aspects rocm This tag is for PRs from ROCm team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants