Skip to content
Snippets Groups Projects

Add functorcollection for vertex isolation

Merged Tommaso Fulghesu requested to merge tfulghes-vertexisolation-functorcollection into master

Follows Rec!3090 (merged)

Functorcollection aiming at replacing the old tuple tool TupleToolVtxIsoln.cpp for the new FunTuple framework.

Base the functorcollection on VertexAndConeIsolation tool defined in Rec and definition of

  • ParticleIsolation (charged OR neutral)
  • ConeIsolation (charged AND neutral + MAXCONE and MINCONE observables)
  • VertexIsolation

Variables added by ParticleIsolation: SUMCONE, ASYM, CMULT, DETA(array) and DPHI(array)

Variables added by VertexIsolation: NTRACKS, SMALLEST_CHI2, SMALLEST_DELTACHI2, MASS_SMALLEST_DELTACHI2

Work towards completion of DPA task https://gitlab.cern.ch/lhcb-dpa/project/-/issues/178.

Edited by Tommaso Fulghesu

Merge request reports

Merge request pipeline #6644053 passed

Merge request pipeline passed for 71d1924e

Approval is optional

Merged by Abhijit MathadAbhijit Mathad 1 year ago (Dec 20, 2023 12:48pm UTC)

Merge details

  • Changes merged into master with ea365a62 (commits were squashed).
  • Deleted the source branch.

Pipeline #6644444 passed

Pipeline passed for ea365a62 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 6825c8e6 - Apply 2 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 2bc15c60 - Fix MIN_ENTRY and adjust documentation adding examples

    Compare with previous version

  • added 1 commit

    • 463a2a27 - Retrieve TO side from the relation entry

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • mentioned in issue Rec#396 (closed)

  • Tommaso Fulghesu added 11 commits

    added 11 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • mentioned in issue Rec#272 (closed)

  • Patrick Koppenburg approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading