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
Remove boost and root deps from SimDataFormats/EncodedEventId #33715
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33715/22640
|
A new Pull Request was created by @guitargeek (Jonas Rembser) for master. It involves the following packages: SimDataFormats/EncodedEventId @cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: Build HeaderConsistency BuildI found compilation error when building: Copying tmp/slc7_amd64_gcc900/src/GeneratorInterface/Hydjet2Interface/src/GeneratorInterfaceHydjet2Interface/libGeneratorInterfaceHydjet2Interface.so to productstore area: Leaving library rule at GeneratorInterface/Hydjet2Interface Entering library rule at src/GeneratorInterface/Hydjet2Interface/test >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-05-12-2300/src/GeneratorInterface/Hydjet2Interface/test/Hydjet2Analyzer.cc /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-05-12-2300/src/GeneratorInterface/Hydjet2Interface/test/Hydjet2Analyzer.cc: In member function 'virtual void Hydjet2Analyzer::analyze(const edm::Event&, const edm::EventSetup&)': /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-05-12-2300/src/GeneratorInterface/Hydjet2Interface/test/Hydjet2Analyzer.cc:554:23: error: 'Cos' is not a member of 'TMath' 554 | v2 = TMath::Cos(2 * (phiTrue)); | ^~~ /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-05-12-2300/src/GeneratorInterface/Hydjet2Interface/test/Hydjet2Analyzer.cc:555:23: error: 'Cos' is not a member of 'TMath' 555 | v3 = TMath::Cos(3 * (phiTrue - phi3)); | ^~~ |
Pull request #33715 was updated. @SiewYan, @civanch, @mkirsano, @mdhildreth, @rekovic, @agrohsje, @alberto-sanchez, @cecilecaillol, @GurpreetSinghChahal can you please check and sign again. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33715/22661
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0b47c2/15121/summary.html CMS Clang-Tidy warnings: There are Clang-Tidy warnings. See https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0b47c2/15121/llvm-analysis/cmsclangtidy.txt for details. Comparison SummarySummary:
|
+l1 |
@guitargeek , it seems FastSim is affected by this clean-up. @sbein , can you, please, have a look? |
Hi @sbein, are these changes OK for you? |
+1 observed difference likely connected with small differences in computation of cosine. |
Thanks! What about generators and upgrade? @agrohsje, @alberto-sanchez, @kpedro88, @srimanob. |
+generators |
+Upgrade Technical PR to remove boost and root dependency. Small changes have been observed due to the calculation of trigonometry function. |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Some dependency cleaning. No backport intended.