Multithreading for cscofflinemonitor 2

[ptcox]

I think I finally have the multithreading interface added to CSCOfflineMonitor in DQMOffline/Muon, for 72X. It also has some fixes for postls1 compatibility.

[cmsbuild]

A new Pull Request was created by @ptcox for CMSSW_7_2_X.

Multithreading for cscofflinemonitor 2

It involves the following packages:

DQMOffline/Muon

@ojeda, @danduggan, @rovere, @cmsbuild, @nclopezo, @deguio, @Degano can you please review it and eventually sign? Thanks.
@barvic, @rociovilar this is something you requested to watch as well.
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.

[cmsbuild]

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

[deguio]

hello @ptcox
this PR is ok for me, thanks. could you comment about how the efficiency histograms are handled now?

cheers,
F.

[ptcox]

Hi Federico,

I haven’t done anything about calculating the efficiencies yet. I was hoping to be able to use DQMGenericClient but I haven’t started working on it. (Since the efficiencies in the old code were broken anyway, we are no worse off now than we were before, and when I find time I’ll try to get the calculation working properly.)

Regards, Tim

On Jun 12, 2014, at 17:13 , deguio notifications@github.com wrote:

hello @ptcox
this PR is ok for me, thanks. could you comment about how the efficiency histograms are handled now?

cheers,
F.

—
Reply to this email directly or view it on GitHub.

[deguio]

+1
thanks for the clarification.
cheers,
F.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine).

[rociovilar]

+1
rocio
On 04 Jun 2014, at 19:42, cmsbuild <notifications@github.commailto:notifications@github.com> wrote:

A new Pull Request was created by @ptcoxhttps://github.com/ptcox for CMSSW_7_2_X.

Multithreading for cscofflinemonitor 2

It involves the following packages:

DQMOffline/Muon

@ojedahttps://github.com/ojeda, @dandugganhttps://github.com/danduggan, @roverehttps://github.com/rovere, @cmsbuildhttps://github.com/cmsbuild, @nclopezohttps://github.com/nclopezo, @deguiohttps://github.com/deguio, @deganohttps://github.com/degano can you please review it and eventually sign? Thanks.
@barvichttps://github.com/barvic, @rociovilarhttps://github.com/rociovilar this is something you requested to watch as well.
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.

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