[14_1_X] Bumped CepGen version to 1.2.3 #9122
Conversation
|
A new Pull Request was created by @forthommel for branch IB/CMSSW_14_1_X/master. @iarspider, @smuzaffar, @cmsbuild, @aandvalenzuela can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
forthommel
changed the title
|
please test |
|
-1 Failed Tests: Build 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: BuildI found compilation error when building: >> Package GeneratorInterface/CepGenInterface built
Entering library rule at src/GeneratorInterface/CepGenInterface/plugins
>> Compiling edm plugin src/GeneratorInterface/CepGenInterface/plugins/CepGenGeneratorFilter.cc
In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_14_1_X_2024-04-06-1100/src/GeneratorInterface/CepGenInterface/interface/CepGenGeneratorFilter.h:8,
from src/GeneratorInterface/CepGenInterface/plugins/CepGenGeneratorFilter.cc:5:
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_14_1_X_2024-04-06-1100/src/GeneratorInterface/CepGenInterface/interface/CepGenEventGenerator.h:36:19: error: 'ParametersList' in namespace 'cepgen' does not name a type
36 | const cepgen::ParametersList proc_params_;
| ^~~~~~~~~~~~~~
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_14_1_X_2024-04-06-1100/src/GeneratorInterface/CepGenInterface/interface/CepGenEventGenerator.h:37:48: error: 'ParametersList' is not a member of 'cepgen'
37 | std::vector > modif_modules_, output_modules_;
| ^~~~~~~~~~~~~~
|
|
cms-sw/cmssw#44647 should fix this issue. Backports to |
|
test parameters: |
|
please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-af56af/38656/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:
|
cmsbuild
added
tests-approved
tests-pending
and removed
tests-started
tests-approved
labels
|
please test for el8_aarch64_gcc12 |
|
please test |
|
-1 Failed Tests: UnitTests 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: Unit TestsI found 1 errors in the following unit tests: ---> test runtestPhysicsToolsPatAlgos had ERRORS |
|
-1 Failed Tests: UnitTests 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: Unit TestsI found 2 errors in the following unit tests: ---> test TestGeneratorInterfacePythia8ConcurrentGeneratorFilter had ERRORS ---> test runtestPhysicsToolsPatAlgos had ERRORS Comparison SummaryThere are some workflows for which there are errors in the baseline: Summary:
|
|
Hello @smuzaffar! |
|
+externals unit tests failure are not related to this update ( these tests are failing for cmssw IBs) @forthommel , I just wanted to make sure new version builds for all supported archs. |
cmsbuild
added
externals-approved
fully-signed
and removed
externals-pending
pending-signatures
labels
|
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_14_1_X/master IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
Many thanks for your detailed explanation, @smuzaffar! I am reassured that this single extra include line is not breaking all the future workflows :-) |
|
please test lets refresh the tests |
|
@smuzaffar This |
@forthommel this one is known: cms-sw/cmssw#33084 (comment) |
|
-1 Failed Tests: UnitTests RelVals-INPUT 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: Unit TestsI found 1 errors in the following unit tests: ---> test runtestPhysicsToolsPatAlgos had ERRORS RelVals-INPUTThe relvals timed out after 4 hours. Comparison SummarySummary:
|
|
REMINDER @sextonkennedy, @rappoccio, @antoniovilela: This PR was tested with cms-sw/cmssw#44647, please check if they should be merged together |
|
+1 Tests in cmssw PR are passing |
|
merge |
cmsbuild
merged commit ca30f8a
into
cms-sw:IB/CMSSW_14_1_X/master
This PR bumps the version of CepGen to 1.2.3.
Changelog is available here.