-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
remove direct calls of DQMStore, move SusyPostProcessor to DQMHarvester #3984
remove direct calls of DQMStore, move SusyPostProcessor to DQMHarvester #3984
Conversation
A new Pull Request was created by @webermat for CMSSW_7_1_X. remove direct calls of DQMStore, move SusyPostProcessor to DQMHarvester It involves the following packages: DQMOffline/JetMET @ojeda, @danduggan, @rovere, @cmsbuild, @nclopezo, @deguio, @Degano can you please review it and eventually sign? Thanks. |
comparison is OK |
@webermat |
+1 |
…014-05-19-0200 remove direct calls of DQMStore, move SusyPostProcessor to DQMHarvester
leftover calls of DQMStore and beginJob and endJob have been removed from all modules (endRun where not needed as well), the SusyPostProcessor has been move to DQMEDHarvester class.
The code was checked by running the matrix, since we call a value map for the ak4jetid of calojets, which has been added to the RECO sequence only recently. Development started from 7_1_X_2014-5-19-0200