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
Set collisionId in prunedGenParticles (11_2_X) #35471
Conversation
A new Pull Request was created by @mandrenguyen (Matthew Nguyen) for CMSSW_11_2_X. It involves the following packages:
@cmsbuild, @santocch can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@mandrenguyen We closed 11_2_X recently as we thought this is not used anymore. @cms-sw/pdmv-l2 Can you confirm that 11_2_X is still used for productions? |
@qliphy 11_2_X is used for heavy-ion re-miniAOD. If we were starting a new re-miniAOD campaign, it would make sense to migrate to a newer release. However, in this case, we will simply need to rerun a few MC samples. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6a4dce/19455/summary.html Comparison SummarySummary:
|
+1 |
merge |
@mandrenguyen CMSSW_11_2_5_patch1which includes this PR is now available: https://hypernews.cern.ch/HyperNews/CMS/get/relAnnounce/2238.html |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_11_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_1_X is complete. This pull request will be automatically merged. |
PR description:
Backport of #35432
11_2_X was used for the reminiAOD of 2018 PbPb data and MC.
Some of the MC datasets will have to be reminiAOD'd with this fix.
Incidentally, I did not see an 11_2_X IB available.
Presumably there is very little development in this release, but hopefully we can build a new one at some point, although it's not very urgent.