Skip to content

test(miopen): add ffm-quick test filter (ROCM-20705)#9351

Open
SreecharanGundaboluAMD wants to merge 2 commits into
developfrom
users/SreecharanGundaboluAMD/ffm_quick_filter
Open

test(miopen): add ffm-quick test filter (ROCM-20705)#9351
SreecharanGundaboluAMD wants to merge 2 commits into
developfrom
users/SreecharanGundaboluAMD/ffm_quick_filter

Conversation

@SreecharanGundaboluAMD

Copy link
Copy Markdown
Contributor

JIRA ID : ROCM-20705

Motivation

This PR introduces a ffm-quick test category for FFM testing. We will expand FFM test coverage gradually.

Technical Details

test_categories.yaml — two changes:

New ffm-quick category added and is labelled for both ffm-quick and ffm-full dispatch. Timeout set to 7200 s (2 h)`.

Risk Assessment

Risk Level: 🟢 Low

Impacted Components

projects/miopen/test/gtest/test_categories.yaml — CI category definitions

Mitigation Steps

Changes are test-only; no kernel, solver, or library logic is touched.

Test Plan

The ffm-quick category will be validated by manually triggering it using github actions.

Test Result

-[x] Validate that the PR automatically picks up and runs the quick filter in this PR.

Submission Checklist

@therock-pr-bot

therock-pr-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
🌿 Branch Name ✅ Pass
📝 PR Title/Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled
🤖 therock-pr-bot ✅ Pass

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@therock-pr-bot

therock-pr-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@SreecharanGundaboluAMD SreecharanGundaboluAMD changed the title test(miopen): add ffm-quick test filter and Smoke/Standard/Full tiering (ROCM-20705) test(miopen): add ffm-quick test filter (ROCM-20705) Jul 13, 2026

@muralinr muralinr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@randyspauldingamd randyspauldingamd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Several patterns are included, but then excluded verbatim. This makes them look like typos, and we should clarify the intent. I'll mark it Approved to reduce iteration though.

A few options are:

  1. More generic wildcards where appropriate:
# in test_patterns:
  "Smoke/GPU_reference_kernel_fwd_3d_NCDHW_"
# in exclude:
  "Smoke/GPU_reference_kernel_fwd_3d_NCDHW_BFP16_BFP16*"  # ...

2a. delete both if they're all not to be ran.
2b. If you wanted to emphasize "do not run this", it would be fine to leave the excluded mention (with a comment)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants