Reapply lost patch to L1TCaloLayer1 unpacker

[nsmith-]

This is used for DQM and affects hardware monitoring information only.

It looks like 81X is out of sync with 8_0_16, should I wait for it to be in sync to make 81X PR or just cherry-pick this onto an IB?

[cmsbuild]

A new Pull Request was created by @nsmith- (Nicholas Smith) for CMSSW_8_0_X.

It involves the following packages:

EventFilter/L1TXRawToDigi

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

cms-bot commands are list here #13028

[davidlange6]

@nsmith- lets solve the 81x issue first - @mulhearn @rekovic should help. (maybe both are away just now)

[nsmith-]

@davidlange6 I guess what I meant is that other packages (unrelated to this change) were showing changes when I tried to PR this against 81X. So I cherry-picked this commit onto 8_1_0_pre10 and made #15644

[rekovic]

+1

Checked that it has no effects on the L1T emulator unit-tests results.

[rekovic]

please test

[cmsbuild]

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

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar

[cmsbuild]

+1
Tested at: b102eed
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15360/15020/summary.html

[cmsbuild]

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

[davidlange6]

+1