bsunanda:Phase2-gem14 Update the forward muon definition in 2023 Geometry scenario

[bsunanda]

Correct the shortcoming in PR13916 (etaMax file was missing in PR13916)

[cmsbuild]

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

It involves the following packages:

Geometry/CMSCommonData

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

cms-bot commands are list here #13028

[bsunanda]

@cmsbuild Please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 45708fa
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13996/12260/summary.html

[cmsbuild]

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

[ianna]

+1

[cmsbuild]

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

[davidlange6]

Thanks @bsunanda - but could you change the title of this PR to be more descriptive. Thanks

[bsunanda]

@davidlange6 Changed the title

[davidlange6]

+1

[boudoul]

Hi @bsunanda ,
I'm trying to built the phase2 WFs (2023dev and 2023LReco) but it seems that this PR causes problems in the DIGI step for the phase 2 geometries -I put here (*) a snippet of the traceback - could you have a look using this recipe ?

git cms-merge-topic 13923
scramv1 b -j 8
runTheMatrix.py -l 10600

Adding @kpedro88 FYI
Many Thanks

()
#5 0x00007fa284763594 in DDValue::doubles() const () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/lib/slc6_amd64_gcc530/libDetectorDescriptionCore
.so
#6 0x00007fa27cc21c40 in RPCGeometryBuilderFromDDD::buildGeometry(DDFilteredView&, MuonDDDConstants const&) () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-0
4-10-0000/lib/slc6_amd64_gcc530/libGeometryRPCGeometryBuilder.so
#7 0x00007fa27cc22bcc in RPCGeometryBuilderFromDDD::build(DDCompactView const, MuonDDDConstants const&) () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-1
0-0000/lib/slc6_amd64_gcc530/libGeometryRPCGeometryBuilder.so
#8 0x00007fa27cf520bc in RPCGeometryESModule::produce(MuonGeometryRecord const&) () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/lib/slc6_amd64_gc
c530/pluginRPCGeometryESModule.so
#9 0x00007fa27cf53f8c in edm::eventsetup::CallbackProxy<edm::eventsetup::Callback<RPCGeometryESModule, std::shared_ptr, MuonGeometryRecord, edm::eventsetup::CallbackSimpleDecorato
r >, MuonGeometryRecord, std::shared_ptr >::getImpl(edm::eventsetup::EventSetupRecord const&, edm::eventsetup::DataKey const&) () from /cvmfs/cms-ib.cern.ch/201
6-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/lib/slc6_amd64_gcc530/pluginRPCGeometryESModule.so
#10 0x00007fa2972eb7df in edm::eventsetup::DataProxy::get(edm::eventsetup::EventSetupRecord const&, edm::eventsetup::DataKey const&, bool) const () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64
gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/lib/slc6_amd64_gcc530/libFWCoreFramework.so
#11 0x00007fa2972e4d0b in edm::eventsetup::EventSetupRecord::getFromProxy(edm::eventsetup::DataKey const&, edm::eventsetup::ComponentDescription const&, bool) const () from /cvmfs/cms-ib.cern.
ch/2016-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/lib/slc6_amd64_gcc530/libFWCoreFramework.so
#12 0x00007fa2708b64b2 in RPCTechnicalTrigger::beginRun(edm::Run const&, edm::EventSetup const&) () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/li
b/slc6_amd64_gcc530/pluginL1TriggerRPCTechnicalTrigger.so
#13 0x00007fa2972a899b in edm::EDProducer::doBeginRun(edm::RunPrincipal const&, edm::EventSetup const&, edm::ModuleCallingContext const_) () from /cvmfs/cms-ib.cern.ch/2016-16/slc6_amd64_gcc530
/cms/cmssw/CMSSW_8_1_X_2016-04-10-0000/lib/slc6_amd64_gcc530/libFWCoreFramework.so

[kpedro88]

@boudoul @bsunanda - There was some previous discussion about the Phase2 muon geometry here: #13992 (diff)

If it's still causing problems, needs to be sorted out ASAP.