Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • 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 155
    • Merge requests 155
  • 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
  • !28868

Force neutral particle charge if calo extrapolation fails in FCS

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Benedict Tobias Winter requested to merge bwinter/athena:force-neutral-if-caloExtrapolation-fails-in-FCS into 21.0 Dec 13, 2019
  • Overview 19
  • Commits 3
  • Pipelines 3
  • Changes 2

The vector returned by the caloHits function in 'FastCaloSimCaloExtrapolation' was sometimes empty for low energy charged particles (T<64 MeV). This subsequently caused an unanounced segfault.

If the vector is empty the function will now rerun the function setting the particle's charge to be neutral. After this fix the error is no longer observed (tested with a few million low-energy charged particles). The default behaviour of the function is unchanged.

The 'caloHits' vector is not expected to ever be non-zero after this fix, which is supported by the tests above. For now an error is thrown if the caloHits vector should still empty after forcing neutral charge, which will point the user to the problem should it occur. The error handling should in the future be improved using StatusCodes. This was not done now to make the fix available rapidly.

Edited Dec 14, 2019 by Tadej Novak
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: force-neutral-if-caloExtrapolation-fails-in-FCS