Add ECAL Phase 2 digis and TP digis validation - 200X - #51403
Conversation
|
A new Pull Request was created by @thomreis for CMSSW_20_0_X. It involves the following packages:
@BenjaminRS, @civanch, @cmsbuild, @ctarricone, @gabrielmscampos, @kpedro88, @mdhildreth, @quinnanm, @rseidita can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
|
cms-bot internal usage |
|
type ecal |
|
backport of #51400 |
|
please test |
|
+1 Size: This PR adds an extra 28KB to repository Comparison SummarySummary:
|
|
+simulation however, I am not sure if the backport is needed. |
Since we would like to have some samples produced centrally with the ECAL development workflows, having the digis validation as part of it would be very useful to verify the production. |
|
Pull request #51403 was updated. @BenjaminRS, @cmsbuild, @ctarricone, @gabrielmscampos, @quinnanm, @rseidita can you please check and sign again. |
|
please test |
|
Pull request #51403 was updated. @BenjaminRS, @cmsbuild, @ctarricone, @gabrielmscampos, @quinnanm, @rseidita can you please check and sign again. |
|
please test |
|
+1 Size: This PR adds an extra 24KB to repository Comparison SummarySummary:
|
|
Hi @gabrielmscampos can you please sign this PR again? |
|
+dqm |
|
+l1 |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_20_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_20_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @ftenchini, @sextonkennedy, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
PR description:
This PR adds validation histograms for the ECAL Phase 2 digis and TP digis. The validation is activated with the eras
phase2_ecal_develandphase2_ecalTP_devel, respectively.PR validation:
Tested with the ECAL Phase 2 developments workflows 34434.612, 34634.612, 34634.61299. The newly added histograms appear in the DQM ROOT file and are filled.
Backport of #51400 to be used for special productions in the next Phase 2 MC campaign.