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
[10.1.X] Bug fixed in SiStripBadComponentInfo #23555
[10.1.X] Bug fixed in SiStripBadComponentInfo #23555
Conversation
… be read by the SiStripBadComponentInfo (clone was missing)
A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_10_1_X. It involves the following packages: DQM/SiStripMonitorClient @vazzolini, @kmaeshima, @dmitrijus, @cmsbuild, @jfernan2, @vanbesien can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
type bug-fix |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
dear @jfernan2, any objection to signing this backport of #23554? |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_1_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_2_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 tested in Configuration/DataProcessing on 2018B express and prompt |
backport of #23554
Greetings,
this PR implements a bug-fix in SiStripBadComponentInfo.cc where bad fiber histogram never got filled (same a proposed in #23491).
In addition for what concerns the addition of bad components from FED errors, this PR is implements a fix via cloning the
siStripQualityESProducer
and then modifying the attributes instead of just changing the attributes from the imported module.Tested against the regular workflow (non-DQM based) and found perfect agreement.
@suchandradutta @fioriNTU @jandrea @arossi83