Mod highest values

[rrabadan]

The possibility to produce a sorted list of the N modules with the largest digi, cluster, off track cluster occupancy when the corresponding tracker maps are produced is added

[cmsbuild]

A new Pull Request was created by @rrabadan (Raul Rabadan) for CMSSW_7_5_X.

Mod highest values

It involves the following packages:

DQM/SiStripMonitorClient

@cmsbuild, @danduggan, @nclopezo, @deguio can you please review it and eventually sign? Thanks.
@threus, @venturia 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[venturia]

@rrabadan, @threus,
I know it is painful and it sounds like a tax to be payed only by honest citizens, but could you profit of this pull request and add in the README.md file of this package the documentation about the instructions on how to use and configure the plugin SiStripOfflineDQM ? It would be very useful.

                                      ciao

                              Andrea

On May 23, 2015, at 3:21 , cmsbuild <notifications@github.commailto:notifications@github.com>
wrote:

A new Pull Request was created by @rrabadanhttps://github.com/rrabadan (Raul Rabadan) for CMSSW_7_5_X.

Mod highest values

It involves the following packages:

DQM/SiStripMonitorClient

@cmsbuildhttps://github.com/cmsbuild, @dandugganhttps://github.com/danduggan, @nclopezohttps://github.com/nclopezo, @deguiohttps://github.com/deguio can you please review it and eventually sign? Thanks.
@threushttps://github.com/threus, @venturiahttps://github.com/venturia 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@nclopezohttps://github.com/nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

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

[cmsbuild]

-1
@rrabadan This pull request cannot be automatically merged, could you please rebase it?
you can see the log for git cms-merge-topic here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9242/7823/git-merge-result

[rrabadan]

Conflicts fixed.

On May 24, 2015, at 10:11, cmsbuild <notifications@github.commailto:notifications@github.com> wrote:

-1
@rrabadanhttps://github.com/rrabadan This pull request cannot be automatically merged, could you please rebase it?
you can see the log for git cms-merge-topic here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9242/7823/git-merge-result

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

[threus]

pretty ping..

[deguio]

+1

[cmsbuild]

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

[davidlange6]

+1