uMNio data skipping implemented for urgent HLT patch

[Spudmeister]

This patch to the HcalUnpacker.cc is needed for the HCAL laser operations in global

[cmsbuild]

A new Pull Request was created by @Spudmeister for CMSSW_8_1_X.

It involves the following packages:

EventFilter/HcalRawToDigi

@cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[Martin-Grunewald]

For HLT online? Then we'd need a PR for 80X also!

[slava77]

Some more details are needed with probably a link to a presentation (or/and an email with run coordination included with details and when this is needed).

[Spudmeister]

I had intended this pull request to be for 8_0_X, as I based my changes on 8_0_X. Is there a way to choose what release the pull request is for?

[slava77]

On 4/20/16 4:14 PM, Spudmeister wrote:

I had intended this pull request to be for 8_0_X, as I based my changes
on 8_0_X. Is there a way to choose what release the pull request is for?

target branch can not be changed after PR is created.
You will need to close one of the 81 PRs you made and create a new for 80X
There is a drop-down menu (or edit in the URL) of the target release
branch to be used before submission

[slava77]

@cmsbuild please test

@Spudmeister what's the ETA for 80X PR?

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/12526/console

[cmsbuild]

+1
Tested at: 9f195f4
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14158/12526/summary.html

[cmsbuild]

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

[slava77]

+1

for #14158 9f195f4

• HCAL unpacker to skip unpacking calibration laser PIN diode inputs coming in the same FED stream.
• jenkins tests pass and comparisons with baseline show no difference (all inputs so far were apparently with getFormatVersion == 1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1