-
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
Run2-alca82 Change output format for MinBias AlCaReco jobs #19524
Conversation
A new Pull Request was created by @bsunanda for master. It involves the following packages: Calibration/HcalAlCaRecoProducers @ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @davidlange6, @lpernie can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild Please test |
The tests are being triggered in jenkins. |
@cmsbuild Please test |
The tests are being triggered in jenkins. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
'keep *_hltTriggerSummaryAOD_*_*', | ||
'keep *_TriggerResults_*_*', | ||
'keep FEDRawDataCollection_rawDataCollector_*_*', | ||
'keep FEDRawDataCollection_source_*_*', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bsunanda
how much increase in size of event content and eventually dataset we expect by keeping all this information?
Can you elaborate what such "solution" ? |
@cmsbuild Please test |
The tests are being triggered in jenkins. |
The old Data Format used to require 0.21 MB per event and the currently proposed one also needs similar amount (a bit less but not significantly lower). So the overall space requirement will be similar. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@franzoni Do you have further doubts for this PR? |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
merge |
Run2-alca86 Backport corrections to Minimum bias and pedestal o/p commands (#19524)
Remove the Digi collection and add Raw and Trigger information