Skip to content

[rocThrust][Code Coverage][Cherry Pick] Increase rocThrust code coverage#754

Closed
NguyenNhuDi wants to merge 2 commits into
ROCm:release/rocm-rel-7.0from
NguyenNhuDi:zenguyen-rocthrust/increase-code-coverage-cherry-pick-7.0
Closed

[rocThrust][Code Coverage][Cherry Pick] Increase rocThrust code coverage#754
NguyenNhuDi wants to merge 2 commits into
ROCm:release/rocm-rel-7.0from
NguyenNhuDi:zenguyen-rocthrust/increase-code-coverage-cherry-pick-7.0

Conversation

@NguyenNhuDi
Copy link
Copy Markdown
Contributor

Remaking this PR #174 to merge into develop instead of release-staging branch


Remaking this PR ROCm#174 to merge into develop instead of release-staging
branch

---------

Co-authored-by: spolifroni-amd <Sandra.Polifroni@amd.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 21, 2025

Codecov Report

❌ Patch coverage is 87.49150% with 184 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
projects/rocthrust/test/test_complex_various.cpp 85.45% 0 Missing and 96 partials ⚠️
projects/rocthrust/test/test_utils.hpp 39.68% 33 Missing and 5 partials ⚠️
projects/rocthrust/test/test_complex.cpp 92.51% 8 Missing and 20 partials ⚠️
projects/rocthrust/test/test_real_assertions.hpp 60.47% 13 Missing and 4 partials ⚠️
projects/rocthrust/test/test_binary_search.cpp 98.48% 0 Missing and 5 partials ⚠️
Additional details and impacted files
@@                   Coverage Diff                    @@
##           release/rocm-rel-7.0     #754      +/-   ##
========================================================
+ Coverage                 75.10%   83.29%   +8.20%     
========================================================
  Files                       495      436      -59     
  Lines                     44357    37976    -6381     
  Branches                   5686     5143     -543     
========================================================
- Hits                      33310    31632    -1678     
+ Misses                     6839     2341    -4498     
+ Partials                   4208     4003     -205     
Flag Coverage Δ
rocRAND ?
rocThrust 83.29% <87.49%> (+3.56%) ⬆️
Files with missing lines Coverage Δ
...ojects/rocthrust/test/test_adjacent_difference.cpp 78.81% <ø> (ø)
projects/rocthrust/test/test_advance.cpp 61.82% <ø> (ø)
projects/rocthrust/test/test_allocator.cpp 75.38% <ø> (ø)
...s/rocthrust/test/test_allocator_aware_policies.cpp 93.62% <ø> (ø)
projects/rocthrust/test/test_async_copy.cpp 92.73% <ø> (ø)
projects/rocthrust/test/test_async_for_each.cpp 96.30% <100.00%> (ø)
projects/rocthrust/test/test_async_reduce.cpp 85.38% <ø> (ø)
projects/rocthrust/test/test_async_scan.cpp 100.00% <ø> (ø)
projects/rocthrust/test/test_async_sort.cpp 95.42% <ø> (ø)
projects/rocthrust/test/test_async_transform.cpp 89.41% <ø> (ø)
... and 113 more

... and 82 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.

@NguyenNhuDi
Copy link
Copy Markdown
Contributor Author

Closing this PR to reopen from source to trigger CI. New PR here: (#1037)

@NguyenNhuDi NguyenNhuDi closed this Aug 1, 2025
vamovsik pushed a commit that referenced this pull request Aug 8, 2025
…age (#1037)

Reopening cherry pick pr from source to tirgger CI. Original cherry pick
pr here: (#754)
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