Skip to content

[rocRAND][Code Coverage] Increase Code Coverage for rocRAND#409

Merged
NguyenNhuDi merged 10 commits into
ROCm:developfrom
NguyenNhuDi:zenguyen-rocrand/code-coverage-develop
Jul 14, 2025
Merged

[rocRAND][Code Coverage] Increase Code Coverage for rocRAND#409
NguyenNhuDi merged 10 commits into
ROCm:developfrom
NguyenNhuDi:zenguyen-rocrand/code-coverage-develop

Conversation

@NguyenNhuDi
Copy link
Copy Markdown
Contributor

This PR is recreated to merge into develop instead of release staging branch. Previous PR is #173.

@NguyenNhuDi
Copy link
Copy Markdown
Contributor Author

The number of files changed in this PR is lower than the original, because I did not include the files that were modified by clang-format only

umfranzw
umfranzw previously approved these changes Jun 30, 2025
Copy link
Copy Markdown
Contributor

@umfranzw umfranzw left a comment

Choose a reason for hiding this comment

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

Looks good to me, pending CI checks.

@umfranzw
Copy link
Copy Markdown
Contributor

umfranzw commented Jul 2, 2025

/azp run rocRAND

umfranzw
umfranzw previously approved these changes Jul 7, 2025
Copy link
Copy Markdown
Contributor

@stanleytsang-amd stanleytsang-amd left a comment

Choose a reason for hiding this comment

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

lgtm pending ci

@NguyenNhuDi NguyenNhuDi dismissed stale reviews from stanleytsang-amd and umfranzw via d0759c7 July 7, 2025 23:28
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

❌ Your project status has failed because the head coverage (52.16%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #409      +/-   ##
===========================================
- Coverage    52.19%   52.16%   -0.03%     
===========================================
  Files           63       63              
  Lines         7437     7437              
  Branches       799      799              
===========================================
- Hits          3881     3879       -2     
  Misses        3239     3239              
- Partials       317      319       +2     
Flag Coverage Δ
rocRAND 52.16% <ø> (-0.03%) ⬇️

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

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stanleytsang-amd stanleytsang-amd self-requested a review July 11, 2025 15:43
@NguyenNhuDi NguyenNhuDi merged commit af67002 into ROCm:develop Jul 14, 2025
7 of 10 checks passed
assistant-librarian Bot pushed a commit to ROCm/rocRAND that referenced this pull request Jul 14, 2025
[rocRAND][Code Coverage] Increase Code Coverage for rocRAND (#409)

This PR is recreated to merge into develop instead of release staging
branch. Previous PR is #173.
NguyenNhuDi added a commit to NguyenNhuDi/rocm-libraries that referenced this pull request Jul 18, 2025
This PR is recreated to merge into develop instead of release staging
branch. Previous PR is ROCm#173.
shahamed pushed a commit that referenced this pull request Jul 19, 2025
This PR is recreated to merge into develop instead of release staging
branch. Previous PR is #173.
vamovsik pushed a commit that referenced this pull request Aug 8, 2025
…AND (#409) (#1038)

Re opening this cherry-pick PR from source to trigger CI. Original PR
here (#759)
ammallya pushed a commit that referenced this pull request Oct 27, 2025
…s-core-1.5.0

Bump rocm-docs-core from 1.4.0 to 1.5.0 in /docs/sphinx
ammallya pushed a commit that referenced this pull request Oct 28, 2025
…s-core-1.5.0

Bump rocm-docs-core from 1.4.0 to 1.5.0 in /docs/sphinx

[ROCm/rocwmma commit: de4d251]
ammallya pushed a commit that referenced this pull request Nov 5, 2025
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.

4 participants