Draft: Add momentum- and position-based pseudorapidity calculations to CompositeParticle
The CompositeParticle
eta
calculation uses the secondary vertex and PV positions to calculate pseudorapidity. This choice was not clear to users and doesn't give sensible results for non-displaced vertices. It could also provide worse resolution than a momentum based calculation for fully reconstructed particles.
This MR:
- Removes the
CompositeParticle::eta
- Adds
CompositeParticle::eta_position
andCompositeParticle::eta_momentum
- Replaces
eta
witheta_position
andeta_momentum
in all selections. Unless the line authors' intentions were clear, I usedeta_position
for lines selecting partially reconstructed decays andeta_momentum
for lines selecting fully reconstructed decays.
FYI @gligorov
Merge request reports
Activity
Filter activity
assigned to @thboettc
added RTA label
added 626 commits
-
eeb92f53...b4c32658 - 531 commits from branch
2024-patches
- b4c32658...3a871db0 - 85 earlier commits
- b15294c0 - Include BB filter in ODQV
- 62549cfd - Merge branch 'anmorris-odqv-BBfilter' into 'master'
- 31d05b17 - Increase timeouts for Allen TCK tests
- 0d306acb - Update References for: !1856 (merged) based on lhcb-master-mr/12363
- 83df9934 - Saving counters result as monitoring histogram
- 72992dff - Merge branch 'dev_kholoimov_monitoring_counters_histogram' into 'master'
- 4dea6047 - Merge branch 'ref_bot_Allen1856' into 'master'
- 6c96b00a - Merge branch 'increase-timeouts-allen-tck-tests' into 'master'
- 562c46f8 - Add functions for momentum- and position-based pseudorapidity calculations for composite particles
- e11da182 - Fixed formatting
Toggle commit list-
eeb92f53...b4c32658 - 531 commits from branch
Please register or sign in to reply