Skip to content

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

Merged
vamovsik merged 4 commits into
release/rocm-rel-7.0from
zenguyen/NguyenNhuDi/rocthrust-increase-code-coverage-cherry-pick-7.0
Aug 8, 2025
Merged

[rocThrust][Code Coverage][Cherry Pick] Increase rocThrust code coverage#1037
vamovsik merged 4 commits into
release/rocm-rel-7.0from
zenguyen/NguyenNhuDi/rocthrust-increase-code-coverage-cherry-pick-7.0

Conversation

@NguyenNhuDi
Copy link
Copy Markdown
Contributor

Reopening cherry pick pr from source to tirgger CI. Original cherry pick pr here: (#754)

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 1, 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    #1037      +/-   ##
========================================================
+ 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%) ⬆️

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

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 84 files with indirect coverage changes

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

xiaohuguo2023 pushed a commit to xiaohuguo2023/rocm-libraries that referenced this pull request Aug 3, 2025
* Add uniform percision float16 to SDDMM

[ROCm/rocSPARSEcommit: a93555a]
@NguyenNhuDi NguyenNhuDi closed this Aug 5, 2025
@NguyenNhuDi NguyenNhuDi deleted the zenguyen/NguyenNhuDi/rocthrust-increase-code-coverage-cherry-pick-7.0 branch August 5, 2025 15:27
@NguyenNhuDi NguyenNhuDi restored the zenguyen/NguyenNhuDi/rocthrust-increase-code-coverage-cherry-pick-7.0 branch August 5, 2025 15:28
@NguyenNhuDi NguyenNhuDi reopened this Aug 5, 2025
@stanleytsang-amd stanleytsang-amd self-requested a review August 5, 2025 16:22
@vamovsik vamovsik merged commit 9ef867b into release/rocm-rel-7.0 Aug 8, 2025
14 of 15 checks passed
@vamovsik vamovsik deleted the zenguyen/NguyenNhuDi/rocthrust-increase-code-coverage-cherry-pick-7.0 branch August 8, 2025 18:28
assistant-librarian Bot pushed a commit to ROCm/rocThrust that referenced this pull request Aug 8, 2025
[rocThrust][Code Coverage][Cherry Pick] Increase rocThrust
 code coverage (#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.

5 participants