TrackerGeometricDetESModule issue in Phase2 D88 DD4hep (step1, GEN-SIM)

[srimanob]

Hi,

In parallel with #36837

Running out-of-box driver from
CMSSW_12_3_X_2022-01-31-2300 with #36836
runTheMatrix.py --what upgrade -l 39434.911 --wm init

I found

----- Begin Fatal Exception 01-Feb-2022 14:13:59 CET-----------------------
An exception of category 'EventSetupConflict' occurred while
   [0] Calling beginJob
Exception Message:
two EventSetup Producers want to deliver type="GeometricDet" label=""
 from record IdealGeometryRecord. The two providers are 
1) type="TrackerGeometricDetESModule" label="trackerNumberingGeometry"
2) type="TrackerGeometricDetESModule" label="trackerNumberingGeometryDB"
Please either
   remove one of these Producers
   or find a way of configuring one of them so it does not deliver this data
   or use an es_prefer statement in the configuration to choose one.
----- End Fatal Exception -------------------------------------------------

The issue was not there with D77, as we tested in #36837.

FYI @mmusich @emiglior

[cmsbuild]

A new Issue was created by @srimanob Phat Srimanobhas.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[srimanob]

assign geometry, tracker-dpg

[cmsbuild]

New categories assigned: geometry

@cvuosalo,@mdhildreth,@ianna,@Dr15Jones,@makortel,@civanch you have been requested to review this Pull request/Issue and eventually sign? Thanks

[srimanob]

assign trk-dpg

[cmsbuild]

New categories assigned: trk-dpg

@connorpa,@mmusich,@tsusa you have been requested to review this Pull request/Issue and eventually sign? Thanks

[mmusich]

-1

The D88 geometry was introduced by the geometry group in #36264 (@bsunanda FYI)

[srimanob]

OK, understand that @mmusich
Just in case you have a quick idea on how things can be double load. If not, no problem. Thanks.

[mmusich]

Just in case you have a quick idea on how things can be double load. If not, no problem.

I guess copying what is done for D77 if that still works?

[srimanob]

Ah, I think I know now.
We miss definition of dd4hep of D86,D88 in
https://github.com/cms-sw/cmssw/blob/master/Configuration/StandardSequences/python/GeometryConf.py
so, that is why cmsDriver pick the default one which is
process.load('Configuration.StandardSequences.GeometryRecoDB_cff')

[bsunanda]

Please note Phase2 cannot operate with DB. It has to work with XML versions till we have the detector geometry in DB.

…

________________________________ From: Phat Srimanobhas ***@***.*** Sent: 01 February 2022 16:10 To: cms-sw/cmssw Cc: Sunanda Banerjee; Mention Subject: Re: [cms-sw/cmssw] TrackerGeometricDetESModule issue in Phase2 D88 DD4hep (step1, GEN-SIM) (Issue #36849) Ah, I think I know now. We miss definition of dd4hep of D86,D88 in https://github.com/cms-sw/cmssw/blob/master/Configuration/StandardSequences/python/GeometryConf.py so, that is why cmsDriver pick the default one which is process.load('Configuration.StandardSequences.GeometryRecoDB_cff') — Reply to this email directly, view it on GitHub<#36849 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABGMZOSUDVBASPISVDUPDFDUY7ZWTANCNFSM5NJEFIVQ>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[srimanob]

Thanks @bsunanda

Fix in #36851 to allow to load properly.