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
introduce SiPhase2BadStrip format for phase-2 simulation #37397
introduce SiPhase2BadStrip format for phase-2 simulation #37397
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37397/29056
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages:
@malbouis, @civanch, @yuanchao, @ggovi, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @AdrianoDee, @srimanob, @Dr15Jones, @clacaputo, @slava77, @jpata, @tvami, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
CalibTracker/SiPhase2TrackerESProducers/plugins/SiPhase2BadStripConfigurableFakeESSource.cc
Outdated
Show resolved
Hide resolved
CalibTracker/SiPhase2TrackerESProducers/plugins/SiPhase2BadStripConfigurableFakeESSource.cc
Outdated
Show resolved
Hide resolved
CalibTracker/SiPhase2TrackerESProducers/plugins/SiPhase2BadStripConfigurableFakeESSource.cc
Outdated
Show resolved
Hide resolved
CalibTracker/SiPhase2TrackerESProducers/plugins/SiPhase2BadStripConfigurableFakeESSource.cc
Outdated
Show resolved
Hide resolved
CalibTracker/SiPhase2TrackerESProducers/plugins/SiPhase2BadStripConfigurableFakeESSource.cc
Outdated
Show resolved
Hide resolved
CondTools/SiPhase2Tracker/plugins/SiPhase2BadStripChannelBuilder.cc
Outdated
Show resolved
Hide resolved
SLHCUpgradeSimulations/Geometry/python/fakePhase2OuterTrackerConditions_cff.py
Outdated
Show resolved
Hide resolved
It also seems that the unit test is failing |
@cms-sw/upgrade-l2 can you comment what is missing there? I've updated the dict2026Geometry.py file, but seems it's not picked up for dd4hep cases.
|
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test test2026Geometry had ERRORS Comparison SummarySummary:
|
5151503
to
0905a21
Compare
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-df6e37/23558/summary.html Comparison SummarySummary:
|
+1
|
+reconstruction
|
+1 |
@cms-sw/upgrade-l2 please take a look and eventually sign, there are more PRs on this subject coming that depend on this entering the release (cf #37397 (comment)). Thank you! |
+Upgrade This PR introduces "fake" SiStripBadStrip Phase-2 Outer Tracker. There is no bad components are simulated yet, no change is expected on the Phase-2 wfs from this PR. |
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 now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
kind ping for @cms-sw/orp-l2 for any residual comments (see #37397 (comment)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't mind to change it in this PR @mmusich , but since I noticed it while scrolling through the code I took a note
+1 |
PR description:
The purpose of this PR is to adapt the current
SiStripBadStrip
infrastructure to accomodate the needs to simulate bad strips in the PS-s and SS modules of the Phase-2 Outer Tracker.CondFormat
SiStripBadStrip
is mostly fine to be used as a container of the bad strip data, though thedecode
andencode
functions needed to be specialized as the total amount of strips in a phase-2 module (maximally = 2 cols*1016 rows) exceeds the total maximal amount of strips in the Phase-0 Strip tracker modules (6 APV * 128 = 728).DataFormats/SiStripCommon/interface/ConstantsForCondObjects.h
and used in thedecodePhase2
andencodePhase2
methods ofSiStripBadStrip
.SiPhase2OuterTrackerBadStripRcd
), together with a novelESSource
(SiPhase2BadStripConfigurableFakeESSource.cc
) that delivers in said record, randomly generatedSiStripBadStrip
data with a configurable fraction of bad components (flatly distributed over all the detector).ESource
is included in all existing Phase-2 geometries via the configuration fragmentfakePhase2OuterTrackerConditions_cff
, loaded in the various geometries, and it is at the moment configured in such a way that no bad components are simulated.CondTools/SiPhase2Tracker
package, together with unit tests to test the payload writing and reading.This PR is technical and no regressions to existing workflows are expected.
PR validation:
Performed following tests:
cmssw
compiles;scram b runtests
runs fine;runTheMatrix.py -l limited --ibeos
runs fine;if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A
cc: @emiglior @suchandradutta @tsusa