Skip to content

[AMD] Run int4_mm tests only for compatible arch#165630

Closed
k-artem wants to merge 2 commits intopytorch:mainfrom
k-artem:gfx1100_fixes_2
Closed

[AMD] Run int4_mm tests only for compatible arch#165630
k-artem wants to merge 2 commits intopytorch:mainfrom
k-artem:gfx1100_fixes_2

Conversation

@k-artem
Copy link
Contributor

@k-artem k-artem commented Oct 16, 2025

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 16, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures

As of commit e0399f9 with merge base 1009790 (image):

NEW FAILURES - The following jobs have failed:

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

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 16, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: jithunnair-amd / name: Jithun Nair (e0399f9)
  • ✅ login: k-artem / name: Artem Kuzmitckii (5b73a04)

@k-artem
Copy link
Contributor Author

k-artem commented Oct 16, 2025

@pytorchbot label "ciflow/rocm"

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 16, 2025

To add these label(s) (ciflow/rocm) to the PR, please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

pytorchmergebot pushed a commit that referenced this pull request Oct 20, 2025
This should allow us to move gfx1100 workflow to a lower frequency and also allow it to be triggered on PRs via a dedicated label, for any PRs that target Navi fixes such as [this](#165630) or [this](#165625).

Pull Request resolved: #165699
Approved by: https://github.com/jeffdaily
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Oct 21, 2025
This should allow us to move gfx1100 workflow to a lower frequency and also allow it to be triggered on PRs via a dedicated label, for any PRs that target Navi fixes such as [this](pytorch#165630) or [this](pytorch#165625).

Pull Request resolved: pytorch#165699
Approved by: https://github.com/jeffdaily
@jeffdaily
Copy link
Collaborator

@pytorchbot rebase

@jeffdaily jeffdaily added the keep-going Don't stop on first failure, keep running tests until the end label Oct 21, 2025
@pytorchmergebot
Copy link
Collaborator

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

k-artem and others added 2 commits October 21, 2025 18:30
Such tests should be skipped for rest including gfx1100(Navi3x)

Signed-off-by: Artem Kuzmitckii <artem.kuzmitckii@amd.com>
@pytorchmergebot
Copy link
Collaborator

Successfully rebased gfx1100_fixes_2 onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via git checkout gfx1100_fixes_2 && git pull --rebase)

@jeffdaily jeffdaily added the ciflow/rocm-navi31 Trigger "default" config CI on ROCm Navi31 label Oct 21, 2025
@jeffdaily
Copy link
Collaborator

@pytorchbot merge -f "navi3 test changes only, lint is passing. too many failures still to make sense of the signal"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

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

zhudada0120 pushed a commit to zhudada0120/pytorch that referenced this pull request Oct 22, 2025
This should allow us to move gfx1100 workflow to a lower frequency and also allow it to be triggered on PRs via a dedicated label, for any PRs that target Navi fixes such as [this](pytorch#165630) or [this](pytorch#165625).

Pull Request resolved: pytorch#165699
Approved by: https://github.com/jeffdaily
zhudada0120 pushed a commit to zhudada0120/pytorch that referenced this pull request Oct 22, 2025
Such tests should be skipped for rest including gfx1100(Navi3x)

Fixes for CI HUD for gfx1100

Pull Request resolved: pytorch#165630
Approved by: https://github.com/jeffdaily

Co-authored-by: Jithun Nair <37884920+jithunnair-amd@users.noreply.github.com>
k-artem added a commit to ROCm/pytorch that referenced this pull request Dec 16, 2025
Such tests should be skipped for rest including gfx1100(Navi3x)

Fixes for CI HUD for gfx1100

Pull Request resolved: pytorch#165630
Approved by: https://github.com/jeffdaily

Co-authored-by: Jithun Nair <37884920+jithunnair-amd@users.noreply.github.com>
(cherry picked from commit e13580e)
k-artem added a commit to ROCm/pytorch that referenced this pull request Dec 16, 2025
)

Such tests should be skipped for rest including gfx1100(Navi3x)

Fixes for CI HUD for gfx1100

Pull Request resolved: pytorch#165630
Approved by: https://github.com/jeffdaily


(cherry picked from commit e13580e)

Fixes #SWDEV-556531

Co-authored-by: Jithun Nair <37884920+jithunnair-amd@users.noreply.github.com>
k-artem added a commit to ROCm/pytorch that referenced this pull request Dec 16, 2025
…ible arch (pytorch#165630) (#2881)

Cherry-pick of #2880 
Need to resolve conflicts

---------

Co-authored-by: Artem Kuzmitckii <artem.kuzmitckii@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/rocm-navi31 Trigger "default" config CI on ROCm Navi31 keep-going Don't stop on first failure, keep running tests until the end Merged module: inductor open source topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants