-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Lowered cluster charge threshold for pixel L1 #47480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lowered cluster charge threshold for pixel L1 #47480
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47480/43926
|
A new Pull Request was created by @ferencek for master. It involves the following packages:
@antoniovilela, @cmsbuild, @davidlange6, @fabiocos, @jfernan2, @mandrenguyen, @rappoccio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
I wasn't sure if |
@cmsbuild, please test |
I think you should. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47480/43946
|
Pull request #47480 was updated. @antoniovilela, @cmsbuild, @davidlange6, @fabiocos, @jfernan2, @mandrenguyen, @rappoccio can you please check and sign again. |
enable profiling |
please test |
Rebased to CMSSW_15_1_X_2025-03-14-1100. Should be ok now. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47480/44096
|
Pull request #47480 was updated. @antoniovilela, @cmsbuild, @davidlange6, @fabiocos, @kskovpen, @mandrenguyen, @mmasciov, @rappoccio can you please check and sign again. |
@cmsbuild, please test |
@cms-sw/tracking-pog-l2 please sign again |
unhold |
+1 |
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will be automatically merged. |
PR description:
This PR lowers the cluster charge threshold in pixel layer 1 from 4000 to 2000 electrons for era Run3_2025. The change is motivated by results presented in https://indico.cern.ch/event/1492109/#118-radiation-damage-projectio during the January 2025 Tracker Week. While the mentioned change is undergoing TSG and the tracking POG validation, this PR is being created to already start its validation. More info about the ongoing validation will be added here as it become available.
PR validation:
Only configuration-level changes so made sure that
scram b
works and checked that the correct changes are picked up in the following example cmsDriver commandsIf this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
Backport to CMSSW_15_0_X for 2025 data taking will be needed.