Updates on DQMOffline/Trigger for Muons

[sscruz]

Backport from #18968

[cmsbuild]

+1
Tested at: 6cb6a92
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19512/21079/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
fc51b5b
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19512/21079/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19512/21079/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19512/21079/summary.html

Comparison Summary:

• You potentially added 488 lines to the logs
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 22
• DQMHistoTests: Total histograms compared: 1813506
• DQMHistoTests: Total failures: 44578
• DQMHistoTests: Total nulls: 1729
• DQMHistoTests: Total successes: 1767033
• DQMHistoTests: Total skipped: 166
• DQMHistoTests: Total Missing objects: 0
• Checked 90 log files, 14 edm output root files, 22 DQM output files

[dmitrijus]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_9_3_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[fwyzard]

hold

[fwyzard]

@sscruz can you check if this PR is included #19961 ?

[cmsbuild]

Pull request has been put on hold by @fwyzard
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

[mtosi]

@sscruz can you please check if the code in this PR is already included in #19961 ?
thanks

[sscruz]

Hi @mtosi, I just checked and my code seems to be in place in #19961. However, I'm on holidays so I haven't been able to run #19961 to see if I still everything is still working properly.
Sorry about this,
Sergio

[mtosi]

enjoy your holidays ! please, let us update as soon as you are back ;)

…

On Tue, Aug 8, 2017 at 9:56 AM, sscruz ***@***.***> wrote: Hi @mtosi <https://github.com/mtosi>, I just checked and my code seems to be in place in #19961 <#19961>. However, I'm on holidays so I haven't been able to run #19961 <#19961> to see if I still everything is still working properly. Sorry about this, Sergio — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEt583dWBocsvzKw7wD1hn5CHpGn9QkDks5sWBSbgaJpZM4OMCLB> .

[sscruz]

hi @mtosi , I managed to run #19961 and all the histograms introduced by this PR seem to be in place.
Sorry this took a bit!

[mtosi]

thanks mia

…

On Thu, Aug 17, 2017 at 12:38 PM, sscruz ***@***.***> wrote: hi @mtosi <https://github.com/mtosi> , I managed to run #19961 <#19961> and all the histograms introduced by this PR seem to be in place. Sorry this took a bit! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19512 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEt58xR5Y_oescEOsCFP-Tdjn4YzS6T1ks5sZBgvgaJpZM4OMCLB> .

[mtosi]

@sscruz
let me cross check
your developments are in #19961
and #19961 is in #20620, which is close to be merged (hopefully)

=> shall we close this PR ?

[sscruz]

hi @mtosi ,
yes, that is correct!

[mtosi]

then, please
@sscruz @davidlange6 could we please close this PR ?