add tracking hit effiicency vs BX on relval (data workflows only)

[mtosi]

this update adds the hit efficiency vs BX on data relval workflows (and all PDs)
@rovere @VinInn

[cmsbuild]

A new Pull Request was created by @mtosi (mia tosi) for CMSSW_8_1_X.

It involves the following packages:

DQM/TrackingMonitorSource
DQMOffline/Configuration

@cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@makortel, @threus, @fioriNTU, @rociovilar 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

[slava77]

@mtosi
what's the memory increase from this?
3564 bins x the number of different types of histograms can become large.

[mtosi]

citing @rovere (thanks)
1 histo w/ 3600 bins --> 3600 x 8bytes ~ 4k x 8 = 32 kB
there are 1 histo per partitions x (type of hit + eff)
there are 27 partitions => 135 histos + 27 histos = 162 histos
=> ~5MB

[slava77]

Sumw2 is never called, right?
it will add 16 bytes per bin.

[mtosi]

=> 15MB ?

[rovere]

Not locally, not sure globally (i.e. via static member functions, e.g. https://root.cern.ch/doc/master/TH1_8cxx_source.html#l06247)

[slava77]

On 6/17/16 8:17 AM, mia tosi wrote:

=> 15MB ?

sounds small.
I suppose there is only one instance of the monitoring module doing it.
Please confirm.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#14915 (comment), or
mute the thread
https://github.com/notifications/unsubscribe/AEdcbmNc2opT70oc8ck47YUPYEl3NrZEks5qMrpugaJpZM4I4XMD.

[rovere]

One instance per stream, you mean, right?
I let Mia comment for how many selected-track-collection we do have the monitoring enabled, since those counts separately as instances, if that's what you are asking for.

[mtosi]

generalTracks and HP tracks w/ pT > 1 GeV
if tracking POG agrees, I can limit these plots to HP w/ pT > 1

[rovere]

Ok, for vs BX enable only HP, pt >1

[mtosi]

I'm going to update the PRs, thus

[cmsbuild]

Pull request #14915 was updated. @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please check and sign again.

[cmsbuild]

Pull request #14915 was updated. @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please check and sign again.

[VinInn]

@cmsbuild , please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 41a8741
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14915/13584/summary.html

[cmsbuild]

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

[dmitrijus]

+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