RPC DQM occupancy plot bug fix for 73x

[acimmino]

RPC DQM occupancy plot bug fix for 73x

[cmsbuild]

A new Pull Request was created by @acimmino (Anna Cimmino) for CMSSW_7_3_X.

RPC DQM occupancy plot bug fix for 73x

It involves the following packages:

DQM/RPCMonitorDigi

@cmsbuild, @danduggan, @nclopezo, @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.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[deguio]

ciao anna,
it seems to me that this PR doesn't cover all the changes submitted for 72.
referring to: https://github.com/cms-sw/cmssw/pull/6619/files

is there a reason?
thanks,
F.

[acimmino]

Dear Federico,

You're correct. The reason is that in 72X the RPC client modules are EDAnalyzers. In 73X and higher, the client modules are DQMEDHarvester. Plus, one bug I fixed and submitted in the PR 6619 was already fixed for 73X and higher. That is why, in this pull requests only a couple of lines of code were involved.

Ciao,
Anna

---

From: deguio [notifications@github.com]
Sent: 03 December 2014 17:47
To: cms-sw/cmssw
Cc: Anna Cimmino
Subject: Re: [cmssw] RPC DQM occupancy plot bug fix for 73x (#6763)

ciao anna,
it seems to me that this PR doesn't cover all the changes submitted for 72.
referring to: https://github.com/cms-sw/cmssw/pull/6619/files

is there a reason?
thanks,
F.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/6763#issuecomment-65443956.

[cmsbuild]

+1
Tested at: 9e912ab
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6763/1420/summary.html

[deguio]

+1

[cmsbuild]

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

[davidlange6]

+1

[cmsbuild]

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