Temporary Fix to match Hcal Trig Towers

[vkhristenko]

An inconsistency in the HCAL TrigDetId treatment introduced by
PR #7950/#8103 outside DQM causes mismatch between HCAL TrigTowers from
data and L1 emulator. It will take some time to make a self-consistent
fix.

An urgent quick fix.
ran runTheMatrix.
VK

[cmsbuild]

A new Pull Request was created by @vkhristenko (Viktor Khristenko) for CMSSW_7_4_X.

Temporary Fix to match Hcal Trig Towers

It involves the following packages:

DQM/HcalMonitorTasks

@cmsbuild, @danduggan, @deguio 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[abdoulline]

@deguio, @davidlange6,
please, consider it for the next 745patchX for data processing

[davidlange6]

@abdoulline - is a das failure... as for the PR - is there some more information on this issue reported in hn or a meeting?

[abdoulline]

Not that I know. There were several private (e-mail) threads and oral discussions at HCAL Operations meetings.
Perhaps Viktor could add a link to elog?

[vkhristenko]

@davidlange6 - if you are asking about the reason for PR (what it is aiming
to fix), then a quick mentioning is here
https://indico.cern.ch/event/370943/contribution/1/material/slides/0.pdf on
page 4 "due to failed TP matching"

VK

On Fri, Jun 19, 2015 at 11:44 AM, abdoulline notifications@github.com
wrote:

Not that I know. There were several private (e-mail) threads and oral
discussions at the HCAL Operations meetings.
Probably Viktor could add a link to elog?

—
Reply to this email directly or view it on GitHub
#9669 (comment).

[abdoulline]

No elog entries?

(in slide 4 - there was incorrect mentioning of LUTs in the context of the TP mismatch)

On Fri, 19 Jun 2015, Viktor Khristenko wrote:

@davidlange6 - if you are asking about the reason for PR (what it is aiming
to fix), then a quick mentioning is here
https://indico.cern.ch/event/370943/contribution/1/material/slides/0.pdf on
page 4 "due to failed TP matching"

VK

On Fri, Jun 19, 2015 at 11:44 AM, abdoulline notifications@github.com
wrote:

Not that I know. There were several private (e-mail) threads and oral
discussions at the HCAL Operations meetings.
Probably Viktor could add a link to elog?

—
Reply to this email directly or view it on GitHub
#9669 (comment).

—
Reply to this email directly or view it on
GitHub.[AEx02ryLiqz7GmZXUEz8y9JojX0JPcfVks5oU_ObgaJpZM4FGPT-.gif]

[deguio]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs once checked with relvals in the development release cycle of CMSSW or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[cmsbuild]

+1
Tested at: 233c3bb
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9669/2621/summary.html

[cmsbuild]

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

[davidlange6]

+1