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!: Untemplate Vertex #2877

Merged

Conversation

paulgessinger
Copy link
Member

@paulgessinger paulgessinger commented Jan 18, 2024

This PR makes Vertex a concrete type, which is possible after moving input track to be a concrete type erased type.

Part of:

Blocked by:

@paulgessinger paulgessinger added the 🛑 blocked This item is blocked by another item label Jan 18, 2024
@paulgessinger paulgessinger added this to the v32.0.0 milestone Jan 18, 2024
@paulgessinger paulgessinger marked this pull request as draft January 18, 2024 09:53
@github-actions github-actions bot added Component - Core Affects the Core module Infrastructure Changes to build tools, continous integration, ... Component - Examples Affects the Examples module Vertexing labels Jan 18, 2024
@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch 2 times, most recently from 8d1bcd1 to f6de685 Compare January 18, 2024 17:37
Copy link

codecov bot commented Jan 18, 2024

Codecov Report

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

Comparison is base (9b2fd7d) 48.94% compared to head (428592a) 48.99%.
Report is 2 commits behind head on main.

❗ Current head 428592a differs from pull request most recent head f6295d4. Consider uploading reports for the commit f6295d4 to get more accurate results

Files Patch % Lines
...e/include/Acts/Vertexing/IterativeVertexFinder.ipp 36.84% 3 Missing and 9 partials ⚠️
...clude/Acts/Vertexing/AdaptiveMultiVertexFinder.ipp 47.61% 1 Missing and 10 partials ⚠️
...clude/Acts/Vertexing/AdaptiveMultiVertexFitter.ipp 33.33% 2 Missing and 6 partials ⚠️
...include/Acts/Vertexing/FullBilloirVertexFitter.ipp 41.66% 0 Missing and 7 partials ⚠️
Core/include/Acts/Vertexing/TrackAtVertex.hpp 68.75% 4 Missing and 1 partial ⚠️
...Acts/Vertexing/AdaptiveGridDensityVertexFinder.ipp 0.00% 1 Missing and 3 partials ⚠️
...include/Acts/Vertexing/GridDensityVertexFinder.ipp 0.00% 0 Missing and 4 partials ⚠️
...nclude/Acts/Vertexing/TrackDensityVertexFinder.ipp 0.00% 0 Missing and 2 partials ⚠️
Core/include/Acts/Vertexing/Vertex.ipp 90.90% 0 Missing and 2 partials ⚠️
Core/include/Acts/Vertexing/ZScanVertexFinder.ipp 33.33% 0 Missing and 2 partials ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2877      +/-   ##
==========================================
+ Coverage   48.94%   48.99%   +0.05%     
==========================================
  Files         496      496              
  Lines       28913    28938      +25     
  Branches    13710    13704       -6     
==========================================
+ Hits        14151    14179      +28     
- Misses       4878     4881       +3     
+ Partials     9884     9878       -6     

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

@paulgessinger paulgessinger modified the milestones: v32.0.0, v33.0.0 Jan 19, 2024
@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch from f6de685 to 3262297 Compare January 22, 2024 08:26
@github-actions github-actions bot removed the Infrastructure Changes to build tools, continous integration, ... label Jan 22, 2024
@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch from 3262297 to 1410a08 Compare January 22, 2024 13:30
@paulgessinger paulgessinger marked this pull request as ready for review January 22, 2024 13:30
@paulgessinger
Copy link
Member Author

This is ready to review now @andiwand.

@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch from 8f04fd0 to cd370bc Compare January 24, 2024 11:37
andiwand
andiwand previously approved these changes Jan 24, 2024
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.

that's a blessing 🙏

@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch from cd370bc to 035a5b4 Compare February 2, 2024 12:20
@acts-policybot acts-policybot bot dismissed andiwand’s stale review February 2, 2024 12:20

Invalidated by push of 035a5b4

@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch from 035a5b4 to 428592a Compare February 2, 2024 12:22
@paulgessinger paulgessinger force-pushed the refactor/untemplate-vertex branch from 428592a to f6295d4 Compare February 5, 2024 09:47
@paulgessinger
Copy link
Member Author

This is ready now @andiwand

@andiwand andiwand added automerge and removed 🛑 blocked This item is blocked by another item labels Feb 5, 2024
@kodiakhq kodiakhq bot merged commit dfd4356 into acts-project:main Feb 5, 2024
54 checks passed
@github-actions github-actions bot removed the automerge label Feb 5, 2024
@acts-project-service
Copy link
Collaborator

🔴 Athena integration test results [dfd4356]

Build job with this PR failed!

Please investigate the build job for the pipeline!

@acts-project-service acts-project-service added the Breaks Athena build This PR breaks the Athena build label Feb 5, 2024
@paulgessinger paulgessinger deleted the refactor/untemplate-vertex branch February 5, 2024 12:33
kodiakhq bot pushed a commit that referenced this pull request Feb 6, 2024
LaraCalic pushed a commit to LaraCalic/acts that referenced this pull request Feb 10, 2024
This PR makes `Vertex` a concrete type, which is possible after moving input track to be a concrete type erased type.

Part of:
- acts-project#2842 

Blocked by:
- acts-project#2876
LaraCalic pushed a commit to LaraCalic/acts that referenced this pull request Feb 10, 2024
@paulgessinger paulgessinger added Breaks Athena build This PR breaks the Athena build and removed Breaks Athena build This PR breaks the Athena build labels Mar 6, 2024
EleniXoch pushed a commit to EleniXoch/acts that referenced this pull request May 6, 2024
This PR makes `Vertex` a concrete type, which is possible after moving input track to be a concrete type erased type.

Part of:
- acts-project#2842 

Blocked by:
- acts-project#2876
EleniXoch pushed a commit to EleniXoch/acts that referenced this pull request May 6, 2024
asalzburger pushed a commit to asalzburger/acts that referenced this pull request May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaks Athena build This PR breaks the Athena build Component - Core Affects the Core module Component - Examples Affects the Examples module Vertexing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants