HGCal DB Parameters Writer

[ianna]

• HGCal DB parameters writer implemented
• HGCal DB parameters analyzer and test configuration

To write them out locally and read them back:

cmsRun CondTools/Geometry/test/hgcalgeometrywriter.py
cmsRun Geometry/HGCalCommonData/test/hgcalParameters_cfg.py

[cmsbuild]

A new Pull Request was created by @ianna (Ianna Osborne) for CMSSW_8_0_X.

It involves the following packages:

CondCore/GeometryPlugins
CondCore/Utilities
CondFormats/GeometryObjects
CondTools/Geometry
Configuration/Geometry
DataFormats/ForwardDetId
Geometry/CaloGeometry
Geometry/CaloTopology
Geometry/HGCalCommonData
Geometry/HGCalGeometry
Geometry/Records
RecoLocalCalo/HGCalRecProducers
SimCalorimetry/HGCalSimProducers
SimDataFormats/CaloTest
SimG4CMS/Calo

@civanch, @diguida, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @franzoni, @Dr15Jones, @cerminar, @slava77, @ggovi, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @vandreev11, @sethzenz, @makortel, @apfeiffer1, @kpedro88, @lgray, @cseez, @pfs, @tocheng this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

Following commands in first line of a comment are recognized

• +1|approve[d]|sign[ed]: L1/L2's to approve it
• -1|reject[ed]: L1/L2's to reject it
• assign <category>[,<category>[,...]]: L1/L2's to request signatures from other categories
• unassign <category>[,<category>[,...]]: L1/L2's to remove signatures from other categories
• hold: L1/all L2's/release manager to mark it as on hold
• unhold: L1/user who put this PR on hold
• merge: L1/release managers to merge this request
• [@cmsbuild,] please test: L1/L2 and selected users to start jenkins tests
• [@cmsbuild,] please test with cms-sw/cmsdist#<PR>: L1/L2 and selected users to start jenkins tests using externals from cmsdist

[ianna]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10445/console

[cmsbuild]

+1
Tested at: c036a77
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12888/10445/summary.html

[cmsbuild]

Pull request #12888 was updated. @civanch, @diguida, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @franzoni, @Dr15Jones, @cerminar, @slava77, @ggovi, @mmusich, @davidlange6 can you please check and sign again.

[ianna]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10449/console

[slava77]

@ianna
#12862 is already merged. It is included in this PR as well and triggers extra signatures/labels. Please remove the parts already in #12862

[ianna]

@slava77 - yes, it's another branch based on #12862. As soon as an IB is available, I'll rebase this one.

[cmsbuild]

+1
Tested at: 2ec26f0
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12888/10449/summary.html

[cmsbuild]

Pull request #12888 was updated. @civanch, @diguida, @ianna, @mdhildreth, @cmsbuild, @franzoni, @Dr15Jones, @cerminar, @ggovi, @mmusich, @davidlange6 can you please check and sign again.

[mmusich]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10454/console
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10458/console

[cmsbuild]

+1
Tested at: 3318e0d
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12888/10458/summary.html

[cmsbuild]

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

[ggovi]

+1

[mmusich]

+1

[ianna]

+1

[civanch]

+1
In workflow 5.1 (FastSIm for run1) there is a problem in HcalHitsV - many nans, it is likely not directly related

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar