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
DD4Hep Algorithms and Shapes #22936
DD4Hep Algorithms and Shapes #22936
Conversation
ianna
commented
Apr 11, 2018
- Includes DD4Hep: Migrate Run 1 Tracker Algorithms #22643
- Add more CMS-specific shapes
please test |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22936/4322 |
The tests are being triggered in jenkins. |
A new Pull Request was created by @ianna (Ianna Osborne) for master. It involves the following packages: DetectorDescription/DDCMS @cmsbuild, @civanch, @Dr15Jones, @ianna, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
I'm surprised that some of these file have AIDA copyright. Are such files unique to CMS or are they copied from DD4Hep? If so, should we make those an external? |
@ianna , may be having detailed discussion on design of DD4Hep is needed? We may suffer from tails of old thread unsafe frameworks. |
-1 Tested at: 1ed0770 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: runTheMatrix-results/1000.0_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT/step3_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT.log |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
@Dr15Jones - these files are unique to CMS and should be under our control for at least some time allow smooth integration with CMS framework. Markus and I are working on the code together. As you may have noticed, there is a lot of copying back and forth. The CMS Tracker Pixel from run 1 is an example in DD4Hep repository. |
@civanch - I'm confident, we can help with threadsafety of DD4Hep when it is an issue. |
Pull request #22936 was updated. @cmsbuild, @civanch, @Dr15Jones, @ianna, @mdhildreth can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
Add unit test Use MessageLogger for debugging
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22936/4420 |
Pull request #22936 was updated. @cmsbuild, @civanch, @Dr15Jones, @ianna, @mdhildreth can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
-1 Tested at: 64be013 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: ClangBuild
I found a compilation error while trying to compile with clang: >> Compiling /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-04-19-2300/src/DetectorDescription/DDCMS/plugins/DDCMSDetector.cc >> Compiling /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-04-19-2300/src/DetectorDescription/DDCMS/plugins/DDTECModuleAlgo.cc >> Compiling /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-04-19-2300/src/DetectorDescription/DDCMS/plugins/DDTIBLayerAlgo.cc >> Compiling /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-04-19-2300/src/DetectorDescription/DDCMS/plugins/DDTrackerAngular.cc In file included from /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-04-19-2300/src/DetectorDescription/DDCMS/test/DDUnits.cppunit.cc:3: /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-04-19-2300/src/DetectorDescription/DDCMS/interface/DDUnits.h:8:23: error: constexpr variable '_pi' must be initialized by a constant expression constexpr long double _pi = std::acos(-1); ^ ~~~~~~~~~~~~~ /cvmfs/cms-ib.cern.ch/nweek-02520/slc6_amd64_gcc630/external/gcc/6.3.0/lib/gcc/x86_64-unknown-linux-gnu/6.3.0/../../../../include/c++/6.3.0/cmath:123:14: note: subexpression not valid in a constant expression { return __builtin_acos(__x); } ^ |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
Replaced by #23037 |