Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: New DetectorNavigator tests and some fixes #2988

Merged
merged 16 commits into from
Mar 6, 2024

Merge branch 'main' into detector-navigation-tests

b4b6d9a
Select commit
Loading
Failed to load commit list.
Merged

refactor: New DetectorNavigator tests and some fixes #2988

Merge branch 'main' into detector-navigation-tests
b4b6d9a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 5, 2024 in 1s

48.68% (+0.05%) compared to 61b5ca0

View this Pull Request on Codecov

48.68% (+0.05%) compared to 61b5ca0

Details

Codecov Report

Attention: Patch coverage is 43.75000% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 48.68%. Comparing base (61b5ca0) to head (b4b6d9a).

Files Patch % Lines
Core/include/Acts/Navigation/DetectorNavigator.hpp 46.15% 4 Missing and 10 partials ⚠️
...nclude/Acts/Navigation/NavigationStateUpdaters.hpp 20.00% 0 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2988      +/-   ##
==========================================
+ Coverage   48.63%   48.68%   +0.05%     
==========================================
  Files         493      493              
  Lines       29043    29053      +10     
  Branches    13849    13854       +5     
==========================================
+ Hits        14124    14144      +20     
+ Misses       4971     4963       -8     
+ Partials     9948     9946       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.