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
New attempt to backport #2910 and #3023 to 70x #3121
New attempt to backport #2910 and #3023 to 70x #3121
Conversation
…object to APVCyclePhaseProducerFromL1TS
A new Pull Request was created by @venturia for CMSSW_7_0_X. New attempt to backport #2910 and #3023 to 70x It involves the following packages: CalibTracker/SiStripESProducers @apfeiffer1, @diguida, @danduggan, @rovere, @monttj, @cmsbuild, @nclopezo, @rcastello, @deguio, @ggovi, @vadler, @Degano, @ojeda can you please review it and eventually sign? Thanks. |
+1 |
A clarification: this back port is really needed for the April global run otherwise some strip tracker DQM plots and the strip bad channel prompt calibration results will be affected |
+1 |
+1 |
It's not orp signed. You should have brought it up in ORP or at least replied to the hook. |
Thanks for the reply. Sorry, we missed the hook, but we tried to mention this 70X backport in a mail thread, in reply to ORP cancellation (given there was no ORP this week). Would still be possible to have this in 70X? Thanks. |
+1 I did not see the git thread yesterday - we can look at building a release tomorrow |
@davidlange6 thanks! |
And for the record - we made a patch1 to include this code for the agr. On Apr 3, 2014, at 9:23 PM, "Roberto Castello" <notifications@github.commailto:notifications@github.com> wrote: @davidlange6https://github.com/davidlange6 thanks! — |
…ttempt New attempt to backport cms-sw#2910 and cms-sw#3023 to 70x
This time it should be cleaner. Be careful: #3023 has not been approved for 71x yet.