Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DQM patch set 749 with caloLayer1Monitor #10862

Conversation

nsmith-
Copy link
Contributor

@nsmith- nsmith- commented Aug 19, 2015

For DQM only

Generated on top of CMSSW_7_4_9 by:

git cms-merge-topic 10381
git cms-merge-topic 10761
git cms-addpkg EventFilter/L1TRawToDigi
git apply patchFineGrain.patch # Gian Michele HF bit patch
git apply client.patch # Jingyu DQM client patch for #10381

mcepeda and others added 30 commits July 1, 2015 21:10
+ Delete old, very outdated files
At the request of L1 offline software folks
For the record, I think this could possibly cause issues in
legacy code, due to the emulator previously being called 'rctDigis'
rather than the preferred 'simRctDigis'
Nick Smith and others added 7 commits August 13, 2015 21:21
@cmsbuild
Copy link
Contributor

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

DQM patch set 749 with caloLayer1Monitor

It involves the following packages:

Configuration/StandardSequences
DQM/Integration
DQM/L1TMonitor
DQMOffline/L1Trigger
EventFilter/L1TRawToDigi
EventFilter/RctRawToDigi
L1Trigger/Configuration

The following packages do not have a category, yet:

EventFilter/L1TRawToDigi

@danduggan, @cmsbuild, @franzoni, @deguio, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @batinkov, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @cerati, @threus, @dgulhan, @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.
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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

Jingyu DQM client patch for cms-sw#10381
@cmsbuild
Copy link
Contributor

Pull request #10862 was updated. @danduggan, @cmsbuild, @franzoni, @deguio, @mulhearn, @davidlange6 can you please check and sign again.

@danduggan
Copy link
Contributor

-1
@nsmith- as discussed, this will be covered by #10880

@nsmith- nsmith- closed this Sep 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants