Prevent possible segm violation in TopSingleLeptonHLTOfflineDQM

[perrotta]

Prevent possible segm violation in DQMOffline.Trigger.TopSingleLeptonHLTOfflineDQM

The code is quite convoluted, and certainlly something better can be done.
However, by just allowing for jets the same kind of check on the vector boundary as it is done a few lines above for electrons and muons one can prevent a segmentation violation when the jet selector remove all jets from the vector correctedJets

[cmsbuild]

A new Pull Request was created by @perrotta for CMSSW_7_1_X.

Prevent possible segm violation in TopSingleLeptonHLTOfflineDQM

It involves the following packages:

DQMOffline/Trigger

@ojeda, @danduggan, @rovere, @cmsbuild, @nclopezo, @deguio, @Degano can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[cmsbuild]

-1
When I ran the RelVals I found an error in the following worklfows:
1001.0 step2

runTheMatrix-results/1001.0_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD/step2_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD.log

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

[perrotta]

It looks like having nothing to do with the content of the pull request:

filesFromList
raise Exception("There are not files in input from the file list")
Exception: There are not files in input from the file list

A.

cmsbuild notifications@github.com ha scritto:

-1
When I ran the RelVals I found an error in the following worklfows:
1001.0 step2

runTheMatrix-results/1001.0_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD/step2_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD.log

you can see the results of the tests here:

https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2401/132/summary.html

---

Reply to this email directly or view it on GitHub:
#2401 (comment)

---

This message was sent using IMP, the Internet Messaging Program.

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2401/150/summary.html

[deguio]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?