-
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
Non reproducibility of LETDCTime HCAL DigiTask histograms in PR tests #28367
Comments
A new Issue was created by @fabiocos Fabio Cossutti. @davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign reconstruction,simulation,dqm |
New categories assigned: dqm,reconstruction,simulation @mdhildreth,@jfernan2,@slava77,@andrius-k,@schneiml,@fioriNTU,@perrotta,@kmaeshima,@civanch you have been requested to review this Pull request/Issue and eventually sign? Thanks |
We've agreed with Long on a workaround and he's just submitted it: In the meantime, I've asked our DQM expert (David @DryRun) to try to figure out what might have been a reason of this issue, specific to 2017 HEP17 (QIE11 pilot module) in legacy HE (QIE8). Even if we know and all agreed earlier this year that HCAL DQM (prompt-reco data-oriented, now fully turned=updated toward Run3) back-compatibility is not required, e.g. see here: #27149 |
it looks like this can be signed and closed. |
According to what @abdoulline wrote in #28368 (comment) , there can be still an issue with the HCAL data-specialized DQM (DQM/HcalTasks: DigiTask, where the issue popped up). That specialized DQM is not even supposed to be used for 2018 data, and of course not for MC and previous years. As such this is certainly not an issue for reco any more, and as far as I understand not even for DQM. |
+1
|
+1 |
After the integration of #28330 we observe non reproducibility of DQM histograms in the LETDCTime section of HCAL DigiTask tests, in PRs which cannot affect this (e.g. #28363).
This happens for test wfs 10024.0, 10042.0 and 10224.0 .
This issue is to keep track of the problem (whose origin seems defined) after the initial remarks by @abdoulline #28330 (comment) until it is solved.
The text was updated successfully, but these errors were encountered: