PCC ntupler 76x - October2015 #11832
Conversation
…or data certification with express stream
|
A new Pull Request was created by @capalmer85 for CMSSW_7_6_X. PCC ntupler 76x - October2015 It involves the following packages: RecoLuminosity/LumiProducer @cmsbuild, @cvuosalo, @slava77 can you please review it and eventually sign? Thanks. |
|
@davidlange6 @slava77 I'd like to get this merged relatively quickly because there are some significant updates that the lumi data certifiers need access to on a short time scale. |
|
@cmsbuild please test |
|
The tests are being triggered in jenkins. |
|
|
||
| if(includeJets){ | ||
| hltjetsToken_=consumes<reco::CaloJetCollection >(edm::InputTag("ak4CaloJets")); | ||
| const int kMaxJetCal = 10000; |
There was a problem hiding this comment.
10k ak4 jets can't fit into CMS acceptance.
This looks excessive regarding memory.
There was a problem hiding this comment.
(I stole this from elsewhere in cmssw, so I didn't look too carefully.)
What value would be acceptable?
There was a problem hiding this comment.
even 100 is plenty enough.
|
@cmsbuild please test |
|
The tests are being triggered in jenkins. |
cmsbuild
added
tests-pending
tests-started
and removed
tests-approved
tests-pending
labels
|
@slava77 when should this be merged? |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
cmsbuild
added
reconstruction-approved
fully-signed
and removed
pending-signatures
reconstruction-pending
labels
|
+1 |
PCC ntupler 76x - October2015
update with BX splitting option;
includeJets (default off);
add .py for data certification with express stream