Skip to content

SVT random noise injection #2670

Open
ShujieL wants to merge 4 commits into
mainfrom
svt-noise-clean
Open

SVT random noise injection #2670
ShujieL wants to merge 4 commits into
mainfrom
svt-noise-clean

Conversation

@ShujieL
Copy link
Copy Markdown
Contributor

@ShujieL ShujieL commented May 18, 2026

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

Inject random sensor noise to SVT (ECTRK, BVTX, BTRK). With provided number of noise hits per layer, the code uses a for loop to randomly select a sensitive module, then a position --> cell ID within that module to inject the noise hit. This mechanism works only if the layer has one uniform module shape and size.

Noise are created at RawHits level, then get combined with real hits before RecHits conversion.

To enable noise injection:

-Pectrk:siendcaptrackernoiserawhits:addnoise=1
-Pbvtx:sibarrelvertexnoiserawhits:addnoise=1
-Pbtrk:sibarrelnoiserawhits:addnoise=1

This PR based on the previous developments at #1988 and #2667. It has relevant changes cherry-picked for rebase, refactoring, and a few more updates to ensure the code serves the purpose.

What is the urgency of this PR?

  • High (please describe reason below)
  • [x ] Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • [x ] New feature (issue #__)
  • Optimization (issue #__)
  • Updated parameters, constants (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR requires changes to geometry (epic PR: __)
  • This PR requires changes to EDM4eic (EDM PR: __)
  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • [x ] AI was used in preparing this PR. Please describe usage below.

CODEX was used for refactoring and debugging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Development

Successfully merging this pull request may close these issues.

1 participant