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: Add angles of incidence in RootMeasurementWriter #2493

Merged
merged 10 commits into from
Oct 2, 2023

Conversation

gagnonlg
Copy link
Contributor

For measurement calibrations, the angles of incidence on a surface are very useful. Previously, the MeasurementReader only saved the truth particle direction in the global frame, so this PR adds the angle of incidence.

To validate, here are the angles of incidence for 1-pixel clusters on the Generic detector barrel:

f_phi

f_theta

(presumably, the phi distribution is asymetric because of the Lorentz angle)

closes #2484

@github-actions github-actions bot added Component - Core Affects the Core module Component - Examples Affects the Examples module Event Data Model labels Sep 29, 2023
andiwand
andiwand previously approved these changes Sep 29, 2023
@andiwand andiwand added this to the next milestone Sep 29, 2023
@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Merging #2493 (294140f) into main (363d111) will decrease coverage by 0.01%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main    #2493      +/-   ##
==========================================
- Coverage   49.81%   49.81%   -0.01%     
==========================================
  Files         466      466              
  Lines       26248    26253       +5     
  Branches    12051    12054       +3     
==========================================
+ Hits        13075    13077       +2     
  Misses       4606     4606              
- Partials     8567     8570       +3     
Files Coverage Δ
Core/include/Acts/Utilities/VectorHelpers.hpp 87.14% <40.00%> (-3.63%) ⬇️

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

@kodiakhq kodiakhq bot merged commit dd3348c into acts-project:main Oct 2, 2023
59 checks passed
@github-actions github-actions bot removed the automerge label Oct 2, 2023
@acts-project-service
Copy link
Collaborator

🔴 Athena integration test results

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 Oct 2, 2023
@gagnonlg
Copy link
Contributor Author

gagnonlg commented Oct 2, 2023

The Athena build failure in question:

/builds/acts/acts-athena-ci/athena/Tracking/Acts/ActsVertexReconstruction/src/IterativePriVtxFinderTool.cxx: In member function 'virtual StatusCode ActsTrk::IterativePriVtxFinderTool::initialize()':
/builds/acts/acts-athena-ci/athena/Tracking/Acts/ActsVertexReconstruction/src/IterativePriVtxFinderTool.cxx:105:16: error: 'struct Acts::IterativeVertexFinder<Acts::FullBilloirVertexFitter<ActsTrk::IterativePriVtxFinderTool::TrackWrapper, Acts::HelicalTrackLinearizer<Acts::Propagator<Acts::EigenStepper<>, Acts::Navigator> > >, Acts::TrackDensityVertexFinder<Acts::FullBilloirVertexFitter<ActsTrk::IterativePriVtxFinderTool::TrackWrapper, Acts::HelicalTrackLinearizer<Acts::Propagator<Acts::EigenStepper<>, Acts::Navigator> > >, Acts::GaussianTrackDensity<ActsTrk::IterativePriVtxFinderTool::TrackWrapper> > >::Config' has no member named 'maximumChi2cutForSeeding'; did you mean 'maximumChi2CutForSeeding'?
  105 |   finderConfig.maximumChi2cutForSeeding = m_maximumChi2cutForSeeding;
      |                ^~~~~~~~~~~~~~~~~~~~~~~~
      |                maximumChi2CutForSeeding
      

Doesn't seem to be caused by this PR

@paulgessinger
Copy link
Member

It's due to #2485, and should be fixed with #2495

@paulgessinger paulgessinger modified the milestones: next, v30.1.0 Oct 5, 2023
kodiakhq bot pushed a commit that referenced this pull request Oct 19, 2023
As stated in #2493, angles of incidence are more useful than global frame direction for calibration purposes, so in the PR we add feed these to the neural calibrator instead.

depends on #2493
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 Changes Performance Component - Core Affects the Core module Component - Examples Affects the Examples module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RootMeasurementWriter does not save angles of incidence
4 participants