Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 181
    • Merge requests 181
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !25491

Closed
Created Aug 08, 2019 by Thomas Strebler@tstrebleDeveloper
  • Report abuse
Report abuse

xAOD::TrackParticle Covariance Matrix Evolution, 21.2 branch (2019.08.08.)

  • Overview 174
  • Commits 44
  • Pipelines 30
  • Changes 28

This is meant as a replacement for !24240 (closed) and has been built on top of it.

Following the discussions with the tracking and reconstruction experts, this is an attempt at implementing a schema evolution for xAOD::TrackParticle. In the new setup the covariance matrix of the defining parameters end up in two separate variables called "definingParametersCovMatrixDiag" and "definingParametersCovMatrixOffDiag". Making it possible to handle the diagonal and off-diagonal elements of the covariance matrix more easily in the I/O system.

The stored elements in "definingParametersCovMatrixOffDiag" are not directly the off-diagonal elements of the covariance matrix but rather the correlation coefficients between -1 and 1, which are better suited for subsequent lossy compression.

The xAODMaker::TrackParticleCompressorTool is also modified to optionally drop some of the off-diagonal elements (assumed to be uncorrelated). A check requiring a positive covariance matrix determinant has also been added, as negative values were found to cause errors while rerunning FTAG vertex refitting in derivations.

Pinging @akraszna and @christos for them to comment if needed

Edited Sep 18, 2019 by Thomas Strebler
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: TrackParticleCovMatrix-OffDiagCorr-21.2-20190611