-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
ECAL for Time dependent MC. Premix option C. Handle for APD/PN tag for time dependence #28214
ECAL for Time dependent MC. Premix option C. Handle for APD/PN tag for time dependence #28214
Conversation
The code-checks are being triggered in jenkins. |
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28214/12331
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
…m:amassiro/cmssw into amassiro_premix_OptionC_2tags_ECAL_11_0_X
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28214/12332
|
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
alca: @christopheralanwest @franzoni @tlampen @pohsun @tocheng |
+upgrade |
+1 |
+1 |
+1 |
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. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
First implementation of the ECAL run dependent MC
Similar to #27970, but that was in 10_6_X
Similar to #27974, but after an email exchange with db experts some of the issues from previous PR have been fixed.
To be tested it needs this PR: #27649
Physics details in the presentation at ECAL DPG here: https://indico.cern.ch/event/847097/
This PR will affect only the mixing step, when the premixed library is mixed with the hard scattering. The idea is that the pileup response, that has been simulated with a given transparency for ECAL crystals, will be scaled to match the transparency of the corresponding IOV.
A new object is introduced ( EcalLaserAPDPNRatiosMCRcd ), that has 1 single IOV, thus allowing this to be used as reference for scaling.
Goal: correct simulation of the photostatistics for pileup, when producing run dependent MC with several IOVs (main changes in run dependent MC for ECAL are transparency and noise).
This PR is needed for testing via relval, checking physics performance and possibly imploved data-MC comparison.