-
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
DT DQM fixes for PoA #13523
DT DQM fixes for PoA #13523
Conversation
…g issues with no copy constructor for Single1D container
For 800patch2, include fix for tracker dqm
A new Pull Request was created by @jfernan2 for CMSSW_8_0_X. It involves the following packages: CalibTracker/SiStripCommon @diguida, @boudoul, @franzoni, @cerminar, @fabozzi, @cmsbuild, @srimanob, @deguio, @mmusich, @vanbesien, @hengne, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
ciao @jfernan2 |
ciao @deguio , I'm sorry, I only meant to change code on DQM/DTMonitorModule package, I understand now that this PR involves other packages so I should have submitted them untouched... |
ciao @jfernan2 |
and yes. you need to push an independent PR to 81x. |
Hi, deletion of files in DQM/DTMonitorClient were by accident, please ignore this PR, I'm sorry for the incovenience. I have create a new PR in 81X with only the changes for DQM/DTMonitorModule |
-1 |
-1 |
Needs to follow PR #13399