SUSY VBF trigger monitoring 93X

[tankit]

use #19294 which was merged this morning and add histos for VBF SUSY path

[cmsbuild]

A new Pull Request was created by @tankit (Minsuk Kim) for master.

It involves the following packages:

DQMOffline/Trigger

@vazzolini, @kmaeshima, @dmitrijus, @cmsbuild, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@battibass, @mtosi, @jhgoh, @calderona, @HuguesBrun, @trocino, @rociovilar this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[mtosi]

@tankit , please update the name of this PR (w/ some more clear title ;) )
then, we will need a backport to 92x

@cmsbuild, please test

tracked by #19142

[fwyzard]

@cmsbuild, please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/21177/console Started: 2017/07/05 15:47

[fwyzard]

Mia, "@cmsbuild, please test" has to be the first line of the message, it doesn't work if you put it in the body.

[fwyzard]

@tankit, could you add here the list of triggers being monitored ?

[tankit]

The list of triggers being monitored:
HLT_Mu8_TrkIsoVVL_DiPFJet40_DEta3p5_MJJ750_HTT300_PFMETNoMu60_v*
HLT_Mu10_TrkIsoVVL_DiPFJet40_DEta3p5_MJJ750_HTT350_PFMETNoMu60_v* (backup path)

[mtosi]

:( thanks

…

On 5 Jul 2017, at 15:44, Andrea Bocci ***@***.***> wrote: Mia, ***@***.***, please test" has to be the first line of the message, it doesn't work if you put it in the body. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[cmsbuild]

-1

Tested at: a9e6bff

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

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19565/21177/summary.html

I found follow errors while testing this PR

Failed tests: UnitTests

• Unit Tests:

I found errors in the following unit tests:

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

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

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

[tankit]

Unit Tests failure probably related to fixing int32->uint32 in https://github.com/cms-sw/cmssw/commits/master/DQMOffline/Trigger/python/ObjMonitor_cfi.py

[mtosi]

I'm a little but puzzled by this is it possible that we need to update the PR w/ the most recent fix done by @amagnan ?

…

On Thu, Jul 6, 2017 at 12:29 PM, Minsuk Kim ***@***.***> wrote: Unit Tests failure probably related to fixing int32->uint32 in https://github.com/cms-sw/cmssw/commits/master/DQMOffline/Trigger/python/ ObjMonitor_cfi.py — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19565 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEt589d1YiHVYOq2foZC8Ne4VWtmredJks5sLLcGgaJpZM4OONvg> .

[amagnan]

yes it looks like it :( Sorry should have checked with Minsuk when she did the merge topic.

…

On 07/06/2017 12:35 PM, mia tosi wrote: I'm a little but puzzled by this is it possible that we need to update the PR w/ the most recent fix done by @amagnan ? On Thu, Jul 6, 2017 at 12:29 PM, Minsuk Kim ***@***.***> wrote: > Unit Tests failure probably related to fixing int32->uint32 in > https://github.com/cms-sw/cmssw/commits/master/DQMOffline/Trigger/python/ > ObjMonitor_cfi.py > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#19565 (comment)>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/AEt589d1YiHVYOq2foZC8Ne4VWtmredJks5sLLcGgaJpZM4OONvg> > . > — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19565 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEeCzG2ntHqBrdAlOxVEwJLJQ_SBrEsXks5sLLiGgaJpZM4OONvg>.

-- #################################################### Dr. Anne-Marie Magnan CMS

---------------------- --------------------------- 42-3-037, CERN Imperial College, HEP Dept. CH-1210 Geneva The Blackett Laboratory Prince Consort Road London SW7 2AZ (+41)(0)22 76 6 25 46 http://www.cern.ch http://www.imperial.ac.uk #################################################### â à è é ê ë æ î ï ô ö ù û ü ç ####################################################

[mtosi]

hmm, is this PR still needed ?
I see #19585, now
if this one is no longer needed, please close this
(and next time, please try to update the PR instead of creating a new one, thanks)

[tankit]

As Mia asked, I close this because of #19585, which has tests approved and comparison available.