-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Run3-hcx337 Fix a bug for wrong DetID assignment in HCAL - relevenat for Runs 1,2,3,4 #39967
Conversation
type bugfix |
type urgent |
@cmsbuild please test |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39967/32898
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39967/32900
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39967/32901
|
A new Pull Request was created by @bsunanda (Sunanda Banerjee) for master. It involves the following packages:
@civanch, @Dr15Jones, @makortel, @ianna, @mdhildreth, @cmsbuild, @bsunanda can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild Please test |
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, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-61788e/28775/summary.html Comparison SummaryThere are some workflows for which there are errors in the baseline: Summary:
|
As PR test shows the error message with Phase-2 DD4hep workflow is gone. |
+1 |
There's a gazillion changes here. Do we understand them? |
A serious bug in the older version and a fix of that bug in this PR will show zillions of difference showing the earlier bug has serious effect on the outcome of MC predictions
…________________________________
From: Marco Musich ***@***.***
Sent: 04 November 2022 17:11
To: cms-sw/cmssw
Cc: Sunanda Banerjee; Mention
Subject: Re: [cms-sw/cmssw] Run3-hcx337 Fix a bug for wrong DetID assignment in HCAL - relevenat for Runs 1,2,3,4 (PR #39967)
Reco comparison results: 46895 differences found in the comparisons
There's a gazillion changes here. Do we understand them?
—
Reply to this email directly, view it on GitHub<#39967 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABGMZOT2HNNN5VOQNQMW2ELWGUYUTANCNFSM6AAAAAARVVY34U>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Does the change impact simulation history? |
It does not change the history in SIM step but surely changes history in DIGI step due to use of thresholds applying to different channels
…________________________________
From: Matti Kortelainen ***@***.***
Sent: 04 November 2022 17:36
To: cms-sw/cmssw
Cc: Sunanda Banerjee; Mention
Subject: Re: [cms-sw/cmssw] Run3-hcx337 Fix a bug for wrong DetID assignment in HCAL - relevenat for Runs 1,2,3,4 (PR #39967)
Does the change impact simulation history?
—
Reply to this email directly, view it on GitHub<#39967 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABGMZORLU36IKBUXJ4Z7ZJDWGU3SDANCNFSM6AAAAAARVVY34U>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Thanks @bsunanda. Then large number of differences in small-scale test is expected, right? (and the statistical compatibility can be assessed only with much larger samples) |
@bsunanda @civanch it looks like there is a change in GEANT history.
Otherwise this particular test looks very much similar to other cases (previously considered) when GEANT history is changed. |
Run3-hcx339 BugFix for HcalDetId in the Sim step- relevant for runs 1,2,3,4 - backport of #39967 to CMSSW version 12_5_X
PR description:
Fix a bug for wrong DetID assignment in HCAL - relevenat for Runs 1,2,3,4
PR validation:
Use the runTheMatrix test workflows
If 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
Need to be back ported to all earlier CMSSW versions