-
Notifications
You must be signed in to change notification settings - Fork 186
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
[PY312] Update various base level tools e.g zlib, xz, cmake and more #9410
Conversation
please test |
A new Pull Request was created by @smuzaffar for branch IB/CMSSW_14_2_X/master. @aandvalenzuela, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
cms-bot internal usage |
Pull request #9410 was updated. |
please test for CMSSW_14_2_TF_X/el8_amd64_gcc12 |
please test for CMSSW_14_2_PY312_X/el8_amd64_gcc12 |
-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 TestFWCoreSkeletons had ERRORS Comparison SummarySummary:
|
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8fcd03/41611/summary.html External BuildI found compilation error when building: File "/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc12/external/py3-tensorflow/2.16.1-7f498ea53b2a28aeb5b5e7d621693f56/lib/python3.12/site-packages/tensorflow/lite/python/authoring/authoring.py", line 43, in from tensorflow.lite.python import convert File "/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc12/external/py3-tensorflow/2.16.1-7f498ea53b2a28aeb5b5e7d621693f56/lib/python3.12/site-packages/tensorflow/lite/python/convert.py", line 25, in import shutils ModuleNotFoundError: No module named 'shutils' error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.AfyjSq (%build) RPM build errors: line 42: It's not recommended to have unversioned Obsoletes: Obsoletes: external+tfaot-model-test-simple+1.0.1-4f700e2ecb2c4fecbd024b559b88636f Macro expanded in comment on line 393: %{aot_config}, pointing to the aot config file of the model to compile (required) |
please test for CMSSW_14_2_PY312_X/el8_amd64_gcc12 |
looks good to go in for PY312_X IBs |
This PR updates many base level packages
Used
-fno-fast-math
to buildxz
(see re-evaluate the optimisation flags forliblzma.so
cmssw#41773). Looks like newxz
version5.6.2
is relatively faster than5.2.5
. ROOT bench mark test re-evaluate the optimisation flags forliblzma.so
cmssw#41773 (comment) timings are below (avg of 5 runs)xz 5.2.5 with -Ofast
: 8.251sxz 5.6.2 with -Ofast -fno-fast-math
: 8.1sAlso added missing
BuildRequires
ongmake/autotools
where needed