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

feat: Track level improvements #2012

Merged
merged 9 commits into from
Apr 6, 2023
Merged

Conversation

paulgessinger
Copy link
Member

@paulgessinger paulgessinger commented Apr 3, 2023

This PR does a few things:

  • Add a method appendTrackState to TrackProxy, allowing convenient building of track state chains
  • Add properties nOutliers, nSharedHits, chi2 and nDoF to tracks
  • Adds a helper function that will calculate these values from the track states and store them on the track. That is similar to the already existing function trajectoryState in MultiTrajectoryHelpers, but it stores the results on the tracks themselves.
  • The helper function is called by the 3 fitters and the CKF, so tracks should always have these numbers.

Closes #1999

Blocked by:

@paulgessinger paulgessinger added the 🛑 blocked This item is blocked by another item label Apr 3, 2023
@paulgessinger paulgessinger added this to the next milestone Apr 3, 2023
@codecov
Copy link

codecov bot commented Apr 3, 2023

Codecov Report

Merging #2012 (599a141) into main (76e9db6) will decrease coverage by 0.04%.
The diff coverage is 35.71%.

@@            Coverage Diff             @@
##             main    #2012      +/-   ##
==========================================
- Coverage   49.83%   49.79%   -0.04%     
==========================================
  Files         415      416       +1     
  Lines       23593    23662      +69     
  Branches    10681    10721      +40     
==========================================
+ Hits        11758    11783      +25     
- Misses       4330     4338       +8     
- Partials     7505     7541      +36     
Impacted Files Coverage Δ
...de/Acts/TrackFinding/CombinatorialKalmanFilter.hpp 32.14% <0.00%> (-0.08%) ⬇️
Core/include/Acts/TrackFitting/Chi2Fitter.hpp 36.66% <0.00%> (-0.16%) ⬇️
...re/include/Acts/TrackFitting/GaussianSumFitter.hpp 41.42% <0.00%> (-0.30%) ⬇️
Core/include/Acts/TrackFitting/KalmanFitter.hpp 44.81% <0.00%> (-0.13%) ⬇️
Core/src/EventData/VectorTrackContainer.cpp 33.76% <23.52%> (-2.30%) ⬇️
...re/include/Acts/EventData/VectorTrackContainer.hpp 48.45% <25.00%> (-4.64%) ⬇️
Core/include/Acts/EventData/TrackHelpers.hpp 40.00% <40.00%> (ø)
Core/include/Acts/EventData/TrackProxy.hpp 84.55% <69.23%> (-1.82%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented Apr 4, 2023

📊 Physics performance monitoring for 599a141

Full report
Seeding: seeded, truth estimated, orthogonal
CKF: seeded, truth smeared, truth estimated, orthogonal
IVF: seeded, truth smeared, truth estimated, orthogonal
Ambiguity resolution: seeded, orthogonal
Truth tracking
Truth tracking (GSF)

Vertexing

Vertexing vs. mu
IVF seeded

IVF truth_smeared

IVF truth_estimated

IVF orthogonal

Seeding

Seeding seeded

Seeding truth_estimated

Seeding orthogonal

CKF

CKF seeded

CKF truth_smeared

CKF truth_estimated

CKF orthogonal

Ambiguity resolution

seeded

Truth tracking (Kalman Filter)

Truth tracking

Truth tracking (GSF)

Truth tracking

@paulgessinger paulgessinger removed the 🛑 blocked This item is blocked by another item label Apr 4, 2023
@paulgessinger paulgessinger requested a review from andiwand April 5, 2023 07:19
Copy link

@EdwardMoyse EdwardMoyse left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks!

paulgessinger added a commit to paulgessinger/acts that referenced this pull request Apr 6, 2023
commit 98355e0
Author: Paul Gessinger <[email protected]>
Date:   Tue Apr 4 15:04:58 2023 +0200

    doxygen

commit a162b2d
Author: Paul Gessinger <[email protected]>
Date:   Mon Apr 3 16:25:25 2023 +0200

    fix wrong function

commit 6021ea2
Author: Paul Gessinger <[email protected]>
Date:   Mon Apr 3 16:13:33 2023 +0200

    unit test, append track state functionality

commit 9a25c0a
Author: Paul Gessinger <[email protected]>
Date:   Mon Apr 3 15:58:43 2023 +0200

    some basic unit testing

commit 83aa3e9
Author: Paul Gessinger <[email protected]>
Date:   Mon Apr 3 14:35:38 2023 +0200

    feat: Fitters calculate track quantities

    Adds a central helper function that calculates number of holes,
    outliers, measurements and shared hits based on the type flags, and
    stores them in the track.

commit 2235f82
Author: Paul Gessinger <[email protected]>
Date:   Mon Apr 3 14:21:25 2023 +0200

    refactor: Changce TrackProxy chi2 method name

commit aa6bc1d
Author: Paul Gessinger <[email protected]>
Date:   Fri Mar 31 14:31:08 2023 +0200

    add ndf and chi2 to track container
@kodiakhq kodiakhq bot merged commit c4d019f into acts-project:main Apr 6, 2023
@github-actions github-actions bot removed the automerge label Apr 6, 2023
@paulgessinger paulgessinger deleted the track-calc branch April 17, 2023 15:43
@paulgessinger paulgessinger modified the milestones: next, v25.0.0 Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with track.nMeasurements() in CKF tracks
3 participants