-
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
Update tensorflow to 2.11 #8533
Conversation
A new Pull Request was created by @iarspider for branch IB/CMSSW_13_2_X/master. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
please test |
-1 Failed Tests: Build BuildI found compilation error when building: /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc11/external/eigen/3bb6a48d8c171cf20b5f8e48bfb4e424fbd4f79e-45b239628ec2ac864b5e7c7c452f9135/include/eigen3/Eigen/src/Core/util/IntegralConstant.h(187): warning #1835-D: attribute "__host__" does not apply here Remark: The warnings can be suppressed with "-diag-suppress " /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_13_2_X_2023-05-30-2300/src/HeterogeneousCore/CUDATest/plugins/TestPortableAnalyzer.cc: In member function 'virtual void TestPortableAnalyzer::analyze(const edm::Event&, const edm::EventSetup&)': /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_13_2_X_2023-05-30-2300/src/HeterogeneousCore/CUDATest/plugins/TestPortableAnalyzer.cc:74:98: error: 'all' is not a member of 'Eigen' 74 | << " m @ " << view.metadata().addressOf_m() << " = { ... {" << view[1].m()(1, Eigen::all) << " } ... } \n"; | ^~~ /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_13_2_X_2023-05-30-2300/src/HeterogeneousCore/CUDATest/plugins/TestPortableAnalyzer.cc:74:98: note: suggested alternatives: In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc11/external/eigen/3bb6a48d8c171cf20b5f8e48bfb4e424fbd4f79e-45b239628ec2ac864b5e7c7c452f9135/include/eigen3/Eigen/Core:19, from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_13_2_X_2023-05-30-2300/src/DataFormats/PortableTestObjects/interface/TestSoA.h:4, |
please test with cms-sw/cmssw#41873 |
-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 errors in the following unit tests: ---> test import-downhill had ERRORS ---> test testDownhill had ERRORS ---> test testhep_ml had ERRORS ---> test testTheano had ERRORS and more ... Comparison SummarySummary:
|
Pull request #8533 was updated. |
1 similar comment
Pull request #8533 was updated. |
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 errors in the following unit tests: ---> test import-downhill had ERRORS ---> test import-hep_ml had ERRORS ---> test testDownhill had ERRORS ---> test testhep_ml had ERRORS and more ... Comparison SummaryThere are some workflows for which there are errors in the baseline: Summary:
|
@iarspider , I think you also need to drop the unit tests from cmssw which are using these dropped externals |
please test with cms-sw/cmssw#41950 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d99c19/33138/summary.html Comparison SummarySummary:
|
Remove tests for removed packages (see cms-sw/cmsdist#8533)
Pull request #8533 was updated. |
Pull request #8533 was updated. |
No description provided.