Skip to content

Conversation

@suchandradutta
Copy link
Contributor

PR description:

Updating Phase2 Tracker Digitizer framework to include Bricked Pixel algorithm for the Inner Tracker (IT). The algorithm gets
activated for the phase 2 tracker geometry (T27) containing bricked pixels in TEPX&TFPX, as well as the central 1 (3) rod(s) of TBPX L2 (L3 and 4). The detailed description of the geometry and corresponding changes got included in the PR#34120 . This update do not have any impact on the Outer Tracker digis (verified with Phase2 DQM plots). Attaching a few slides for the IT distributions.
BrickedPR.pdf

PR validation:

Validation done using SingleMuFlatPt2To100 events (runTheMatrix workflow 39104.0). No change observed in Digi DQM distributions for OT, as expected.

Verified that the PR passes the basic test procedure suggested in the CMSSW PR instructions
@emiglior

fwyzard and others added 16 commits July 31, 2021 09:25
…_for_python3

Fix integer division after migration to python 3
…tSetup_filling

Fix uploading EventSetup conditions from multiple CUDA streams [12.0.x]
…nebeammonitor_12_0_X

 Update DQM onlinebeammonitor [12_0_X]
NANO: rework for Prompt (part2) backport
Run3-hcx306X Bug fix (backport cms-sw#34738) to HCAL SD class
…pre5

HLT menu migration to CMSSW 12_0_0_pre5 template [12_0_X]
Fix crash of ECAL GPU reco when ECAL is out of the run - 12_0_X
[Backport] GEM dataformat - fix classversion
…_for_empty_events_120x

Protect HCAL GPU-related modules against empty events [12.0.x]
@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 6, 2021

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35160/25089

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 6, 2021

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35160/25090

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@suchandradutta
Copy link
Contributor Author

Closing the PR. Re-submit it again after resolving the conflicts.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants