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: KalmanVertex(Track)Updater interface change #2955

Conversation

paulgessinger
Copy link
Member

This changes the interface found in KalmanVertexUpdater and KalmanVertexTrackUpdater to handling the dispatch into 3 and 4 dimensions at runtime inside their respective compilation units.

Part of:

Blocked by:

@paulgessinger paulgessinger added the 🛑 blocked This item is blocked by another item label Feb 16, 2024
@paulgessinger paulgessinger added this to the next milestone Feb 16, 2024
@github-actions github-actions bot added Component - Core Affects the Core module Component - Examples Affects the Examples module Vertexing labels Feb 16, 2024
@paulgessinger paulgessinger force-pushed the refactor/vtx-templates-part8-kvu-refactor branch from 0fa0d0e to 49d2a7a Compare February 19, 2024 09:46
@paulgessinger paulgessinger removed the 🛑 blocked This item is blocked by another item label Feb 19, 2024
@paulgessinger paulgessinger force-pushed the refactor/vtx-templates-part8-kvu-refactor branch from 3979a6e to c4f6b1c Compare February 19, 2024 10:40
@paulgessinger paulgessinger marked this pull request as ready for review February 19, 2024 10:40
@github-actions github-actions bot removed the Component - Examples Affects the Examples module label Feb 19, 2024
Copy link

codecov bot commented Feb 19, 2024

Codecov Report

Attention: 81 lines in your changes are missing coverage. Please review.

Comparison is base (148e0e3) 48.82% compared to head (0489b93) 48.78%.

Files Patch % Lines
Core/src/Vertexing/KalmanVertexUpdater.cpp 24.03% 3 Missing and 76 partials ⚠️
...clude/Acts/Vertexing/AdaptiveMultiVertexFitter.ipp 50.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2955      +/-   ##
==========================================
- Coverage   48.82%   48.78%   -0.05%     
==========================================
  Files         496      493       -3     
  Lines       28938    28914      -24     
  Branches    13751    13752       +1     
==========================================
- Hits        14130    14105      -25     
  Misses       4909     4909              
- Partials     9899     9900       +1     

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

@paulgessinger
Copy link
Member Author

Refactored again now @andiwand. I think the duplication of functions should be resolved now, and only two functions remain in the header.

Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

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

awesome!

@kodiakhq kodiakhq bot merged commit 1e6c8b3 into acts-project:main Feb 19, 2024
54 checks passed
@paulgessinger paulgessinger deleted the refactor/vtx-templates-part8-kvu-refactor branch February 19, 2024 20:25
@paulgessinger paulgessinger modified the milestones: next, v33.0.0 Mar 6, 2024
EleniXoch pushed a commit to EleniXoch/acts that referenced this pull request May 6, 2024
…#2955)

This changes the interface found in `KalmanVertexUpdater` and `KalmanVertexTrackUpdater` to handling the dispatch into 3 and 4 dimensions at runtime inside their respective compilation units.

Part of:
- acts-project#2842 

Blocked by:
- acts-project#2948
asalzburger pushed a commit to asalzburger/acts that referenced this pull request May 21, 2024
…#2955)

This changes the interface found in `KalmanVertexUpdater` and `KalmanVertexTrackUpdater` to handling the dispatch into 3 and 4 dimensions at runtime inside their respective compilation units.

Part of:
- acts-project#2842 

Blocked by:
- acts-project#2948
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Core Affects the Core module Vertexing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants