Skip to content
GitLab
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 168
    • Merge requests 168
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !37961

BugFix MM Overlay and sTGC updates

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Alexandre Laurier requested to merge alaurier/athena:NSWDigiUpdates into 21.3 Nov 05, 2020
  • Overview 6
  • Commits 1
  • Pipelines 1
  • Changes 7

2 main focus points of this MR: a bugfix and parameter updates.

Bugfix / temporary hack to the Micromegas overlay algorithm where a seg fault would very rarely occur. In some very busy signal events, the signal container can be larger than the background container and goes agains the basic assumption of the main overlay alg. The seg faults occur when an empty vector is trying to be accessed. For now, skip the overlay with empty MM digits.

The underlying source of this bug was not found. The MM digitization does not produce empty digits; the main overlay alg somehow tries to overlay a digit with a non-existing digit.

Updates:

  1. Update the pileup time window fed into the pileup jobs for the NSW technologies. sTGC values were never properly set, and constrained the MM window to something more reasonable.
  2. Update sTGC charge threshold after deposit charge units were changed in previous MR.
  3. Match sTGC overlay time integration window to better match digitization and expected behaviour.
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: NSWDigiUpdates