Skip to content
Snippets Groups Projects

Draft: Add momentum- and position-based pseudorapidity calculations to CompositeParticle

Open Thomas Boettcher requested to merge thboettc_eta_mom_pos into master

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 and CompositeParticle::eta_momentum
  • Replaces eta with eta_position and eta_momentum in all selections. Unless the line authors' intentions were clear, I used eta_position for lines selecting partially reconstructed decays and eta_momentum for lines selecting fully reconstructed decays.

FYI @gligorov

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading