-
Notifications
You must be signed in to change notification settings - Fork 183
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
Re-add patches to propagate lhe numbering through herwig #8670
Re-add patches to propagate lhe numbering through herwig #8670
Conversation
This reverts commit 7f3bb5c.
A new Pull Request was created by @Dominic-Stafford for branch IB/CMSSW_13_3_X/master. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
test parameters:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d7f3da/34512/summary.html Comparison SummarySummary:
|
+externals this should only be merged with cms-sw/cmssw#42673 |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_13_3_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
please test workflow 535.0,537.0,538.0 |
ah has been merged with this PR. let force merge this too |
6efbd78
into
cms-sw:IB/CMSSW_13_3_X/master
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d7f3da/34639/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
Re-adds the necessary Herwig patches to read numbers from an LHE file, which were initially added in #8349, then reverted in #8417 due to an issue with multithreading.