Skip to content

Conversation

@sbaldu
Copy link
Member

@sbaldu sbaldu commented Dec 2, 2025

No description provided.

@sbaldu sbaldu added API: C++ This change influences the C++ API backend: C++ This only touches the C++ backend feature wip Work in progress labels Dec 2, 2025
@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

❌ Patch coverage is 95.89041% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.11%. Comparing base (e1e5f33) to head (ecebe1c).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...Estering/data_structures/detail/AssociationMap.hpp 88.88% 4 Missing ⚠️
...lude/CLUEstering/core/detail/ClusteringKernels.hpp 98.11% 1 Missing ⚠️
include/CLUEstering/core/detail/SetupTiles.hpp 87.50% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (95.89%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #320      +/-   ##
==========================================
- Coverage   97.21%   97.11%   -0.11%     
==========================================
  Files          53       53              
  Lines        1546     1700     +154     
  Branches       76       89      +13     
==========================================
+ Hits         1503     1651     +148     
- Misses         43       49       +6     
Flag Coverage Δ
unittests 97.11% <95.89%> (-0.11%) ⬇️

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

Files with missing lines Coverage Δ
include/CLUEstering/core/Clusterer.hpp 100.00% <100.00%> (ø)
include/CLUEstering/core/detail/Clusterer.hpp 100.00% <100.00%> (ø)
...ude/CLUEstering/data_structures/internal/Tiles.hpp 100.00% <100.00%> (ø)
...CLUEstering/data_structures/internal/TilesView.hpp 100.00% <100.00%> (ø)
...lude/CLUEstering/core/detail/ClusteringKernels.hpp 98.71% <98.11%> (-0.35%) ⬇️
include/CLUEstering/core/detail/SetupTiles.hpp 97.22% <87.50%> (+0.16%) ⬆️
...Estering/data_structures/detail/AssociationMap.hpp 92.70% <88.88%> (-0.89%) ⬇️

... and 2 files with indirect coverage changes

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

@sbaldu sbaldu force-pushed the feature/batched-clustering branch from d989778 to 2cc2ede Compare December 3, 2025 12:45
@sbaldu sbaldu added this to the 2.10.0 milestone Dec 8, 2025
@sbaldu sbaldu force-pushed the feature/batched-clustering branch 4 times, most recently from 26978c8 to 7c86db4 Compare January 9, 2026 15:30
@sbaldu sbaldu force-pushed the feature/batched-clustering branch 2 times, most recently from 00997d9 to 7685017 Compare January 9, 2026 15:50
Add interface for running from device points
@sbaldu sbaldu force-pushed the feature/batched-clustering branch from 7685017 to 199f58a Compare January 9, 2026 16:01
@sbaldu sbaldu force-pushed the feature/batched-clustering branch from 8889593 to ecebe1c Compare January 14, 2026 11:03
@sbaldu sbaldu merged commit 1c4dc9e into cms-patatrack:main Jan 14, 2026
36 of 39 checks passed
@sbaldu sbaldu deleted the feature/batched-clustering branch January 14, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API: C++ This change influences the C++ API backend: C++ This only touches the C++ backend feature wip Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant