Skip to content

[HipSparseLt] Add CTests for FFM and merge SPMM into other tests#8750

Open
leowu2017 wants to merge 2 commits into
developfrom
users/leowu/rocm-20727
Open

[HipSparseLt] Add CTests for FFM and merge SPMM into other tests#8750
leowu2017 wants to merge 2 commits into
developfrom
users/leowu/rocm-20727

Conversation

@leowu2017

@leowu2017 leowu2017 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

ROCM-20727

Motivation

TheRock CI has changed the ctest matching rule from substring to exact match. Therefore, test_categories.yaml needs to be updated accordingly to ensure consistent behavior.

Technical Details

  1. Combine SPMM and other tests, and keep the same timeout.
  2. Add tests for FFM (ffm_quick, ffm_full), each with a 2-hour timeout.

Test Plan

The tests are triggered as expected.

Test Result

image

Submission Checklist

@leowu2017 leowu2017 requested a review from a team as a code owner June 24, 2026 04:04
@leowu2017 leowu2017 force-pushed the users/leowu/rocm-20727 branch from 93318d6 to f833716 Compare June 24, 2026 04:45
@leowu2017 leowu2017 force-pushed the users/leowu/rocm-20727 branch from f833716 to 9290f79 Compare June 24, 2026 04:46
@leowu2017 leowu2017 changed the title Add CTests for FFM and merge SPMM into other tests [HipSparseLt] Add CTests for FFM and merge SPMM into other tests Jun 24, 2026

@vin-huang vin-huang 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

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.

2 participants