Skip to content
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

Phase2-hgx82 Add the required changes to utilize wafers defined as ExtrudedPolygon #18774

Merged
merged 5 commits into from Jun 9, 2017

Conversation

bsunanda
Copy link
Contributor

@bsunanda bsunanda commented May 16, 2017

Extruded polygon is a recommended shape by G4 team to replace Polyhedra and Boolean utilizing Polyhedra. This is introduced for HGCal geometry

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @bsunanda for master.

It involves the following packages:

Geometry/HGCalCommonData

@civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @kpedro88, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

@bsunanda
Copy link
Contributor Author

@cmsbuild Please test

@cmsbuild
Copy link
Contributor

cmsbuild commented May 16, 2017

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19887/console Started: 2017/05/16 21:30

@cmsbuild
Copy link
Contributor

-1

Tested at: 39f66d9

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18774/19887/summary.html

I found follow errors while testing this PR

Failed tests: Build

  • Build:

I found an error when building:

>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/HGCalCommonData/plugins/HGCalNumberingInitialization.cc 
>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/HGCalCommonData/plugins/HGCalParametersESModule.cc 
>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/HGCalCommonData/plugins/FastTimeNumberingInitialization.cc 
>> Compiling edm plugin /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/HGCalCommonData/plugins/FastTimeParametersESModule.cc 
In file included from /build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/CaloTopology/src/HGCalTopology.cc:1:0:
/build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/CaloTopology/interface/HGCalTopology.h:88:3: error: 'HGCalGeometryMode' does not name a type
   HGCalGeometryMode geomMode() const {return mode_;}
   ^
/build/cmsbld/jenkins-workarea/workspace/ib-any-integration/CMSSW_9_2_X_2017-05-16-1100/src/Geometry/CaloTopology/interface/HGCalTopology.h:134:3: error: 'HGCalGeometryMode' does not name a type
   HGCalGeometryMode           mode_;
   ^


@cmsbuild
Copy link
Contributor

Comparison not run due to Build errors (RelVals and Igprof tests were also skipped)

@cmsbuild
Copy link
Contributor

Pull request #18774 was updated. @civanch, @Dr15Jones, @ianna, @mdhildreth, @dmitrijus, @cmsbuild, @kpedro88, @vanbesien, @davidlange6 can you please check and sign again.

@bsunanda
Copy link
Contributor Author

@cmsbuild Please test

@cmsbuild
Copy link
Contributor

cmsbuild commented May 16, 2017

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19897/console Started: 2017/05/16 22:40

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2017

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/20434/console Started: 2017/06/08 18:03

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2017

Pull request #18774 was updated. @civanch, @Dr15Jones, @ianna, @mdhildreth, @dmitrijus, @cmsbuild, @kpedro88, @vanbesien, @davidlange6 can you please check and sign again.

@bsunanda
Copy link
Contributor Author

bsunanda commented Jun 8, 2017

@ianna @civanch @kpedro88 @dmitrijus @davidlange6 I removed two casting statements as suggested by David. This is a PR waiting former than 3 weeks. Could you approve this at the earliest?

@kpedro88
Copy link
Contributor

kpedro88 commented Jun 8, 2017

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2017

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2017

Comparison job queued.

@ianna
Copy link
Contributor

ianna commented Jun 8, 2017

+1

@civanch
Copy link
Contributor

civanch commented Jun 8, 2017

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2017

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18774/20434/summary.html

Comparison Summary:

  • You potentially added 6 lines to the logs
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 23
  • DQMHistoTests: Total histograms compared: 1867702
  • DQMHistoTests: Total failures: 5290
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 1862239
  • DQMHistoTests: Total skipped: 173
  • DQMHistoTests: Total Missing objects: 0
  • Checked 94 log files, 14 edm output root files, 23 DQM output files

@davidlange6 davidlange6 merged commit 97ed214 into cms-sw:master Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants