Skip to content

[rocsparse] Reduce CI test redundancy while maintaining code coverage#3972

Merged
kliegeois merged 35 commits intoROCm:developfrom
kliegeois:reduced_ci
Jan 25, 2026
Merged

[rocsparse] Reduce CI test redundancy while maintaining code coverage#3972
kliegeois merged 35 commits intoROCm:developfrom
kliegeois:reduced_ci

Conversation

@kliegeois
Copy link
Copy Markdown
Contributor

@kliegeois kliegeois commented Jan 20, 2026

Summary

This PR reduces test parameter combinations in rocsparse's test suite to significantly decrease CI execution time while maintaining or improving code coverage.

Comment thread projects/rocsparse/clients/tests/test_csrmv.yaml
@kliegeois kliegeois changed the title [rocsparse] Reduce CI test redundancy while maintaining code coverage of level 1 and 2 [rocsparse] Reduce CI test redundancy while maintaining code coverage Jan 21, 2026
@ROCm ROCm deleted a comment from codecov-commenter Jan 22, 2026
@ROCm ROCm deleted a comment from codecov-commenter Jan 23, 2026
@jsandham jsandham requested a review from a team as a code owner January 23, 2026 23:20
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #3972       +/-   ##
============================================
+ Coverage    43.62%   69.19%   +25.57%     
============================================
  Files           30      476      +446     
  Lines        11442    72597    +61155     
  Branches      1440    14775    +13335     
============================================
+ Hits          4991    50231    +45240     
- Misses        5984    15907     +9923     
- Partials       467     6459     +5992     
Flag Coverage Δ
hipBLASLt ?
rocSPARSE 69.19% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...sparse/library/src/precond/gtsv/rocsparse_gtsv.cpp 80.69% <ø> (ø)

... and 505 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kliegeois kliegeois merged commit 6670951 into ROCm:develop Jan 25, 2026
25 of 27 checks passed
samjwu added a commit that referenced this pull request Jan 26, 2026
…gs, not just the latest commit (#4081)

## Motivation

<!-- Explain the purpose of this PR and the goals it aims to achieve.
-->

Use carryforward flags so that reports compare between projects or
flags, not just the latest commit

eg: for
#3972 (comment)
the large diff is due to codecov comparing the PR coverage to hipBLASLt
(the last commit to develop) instead of the rocSPARSE flag

## Technical Details

<!-- Explain the changes along with any relevant GitHub links. -->

Uses carryforward as default for flag management and carryfoward flags
for PR comments

https://docs.codecov.com/docs/flags
https://docs.codecov.com/docs/carryforward-flags

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
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