Skip to content

fix(hipsparse): expand SpSM regression testing - #10246

Open
jsandham wants to merge 7 commits into
ROCm:developfrom
jsandham:expand_spsm_testing
Open

fix(hipsparse): expand SpSM regression testing#10246
jsandham wants to merge 7 commits into
ROCm:developfrom
jsandham:expand_spsm_testing

Conversation

@jsandham

@jsandham jsandham commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Motivation

Expands SpSM testing similar to #8565 in order to detect regressions when reusing the sparse matrix descriptor.

Technical Details

A new test, testing_spsm_csr_reuse_descr, exercises the descriptor-reuse paths. For a single CSR sparse-matrix descriptor it sweeps over a set of operations (NON_TRANSPOSE, TRANSPOSE), fill mode (LOWER, UPPER), diag type (UNIT, NON_UNIT) and SpSM algorithms (DEFAULT), validating every result against a host reference. Two call patterns are covered are:

  • call_spsm — runs the full bufferSize → preprocess → SpSM flow repeatedly across configurations, so each unique (op, fill_mode, diag_type, alg) must compute the correct result despite reusing the sparse matrix descriptor.

  • call_spsm_shared_buffer — runs multiple configurations against a single user-provided externalBuffer sized to the maximum required across all configurations, verifying that one shared user buffer remains valid when reusing the sparse matrix descriptor.

JIRA ID: AISPARSE-646

@therock-pr-bot

therock-pr-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass
🔎 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.

🙋 Wish to Override Policy?

@therock-pr-bot

therock-pr-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown

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

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.

1 participant