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

G4e refitter configuration broken by dd4hep #38739

Closed
mmusich opened this issue Jul 14, 2022 · 12 comments
Closed

G4e refitter configuration broken by dd4hep #38739

mmusich opened this issue Jul 14, 2022 · 12 comments

Comments

@mmusich
Copy link
Contributor

mmusich commented Jul 14, 2022

Since there are widespread plans of using the G4e refitting technology for both the muon momentum scale calibration for the W mass analysis and for the global tracker alignment procedure, it's important that we start testing it.
Unfortunately the current setup is broken, possibly from the dd4hep updates in the configuration.
Running the setup put together at PR #38738 results in:

----- Begin Fatal Exception 14-Jul-2022 14:06:57 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Constructing module: class=GeometryProducer label='geopro'
Exception Message:
MissingParameter: Parameter 'g4GeometryDD4hepSource' not found.
----- End Fatal Exception -------------------------------------------------

This was also reported quite some time ago at https://cms-talk.web.cern.ch/t/help-with-running-g4-refit-in-12-4-x/10294 but no replies so far.

@mmusich
Copy link
Contributor Author

mmusich commented Jul 14, 2022

assign geometry, simulation

@cmsbuild
Copy link
Contributor

New categories assigned: geometry,simulation

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

@mmusich
Copy link
Contributor Author

mmusich commented Jul 14, 2022

urgent

@cmsbuild
Copy link
Contributor

A new Issue was created by @mmusich Marco Musich.

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

cms-bot commands are listed here

@VinInn
Copy link
Contributor

VinInn commented Jul 26, 2022

made progress.
try to create geom in beginRun

now I get
----- Begin Fatal Exception 26-Jul-2022 11:39:10 CEST-----------------------
An exception of category 'DetectorDescriptionFault' occurred while
[0] Processing global begin Run run: 1
[1] Calling method for module GeometryProducer/'geopro'
Exception Message:
DDG4SolidConverter::convert: conversion failed for s=ebalgo:ESPM ShapelessSolid: shapeless
solid.shape()=ShapelessSolid
----- End Fatal Exception -------------------------------------------------

@VinInn
Copy link
Contributor

VinInn commented Jul 26, 2022

more progress
with

process.GlobalTag = GlobalTag(process.GlobalTag, 'auto:run1_mc', '')

seems to work (in wf 20.0)

at least

cmsRun Geant4e_example_cfg.py

does not crash

@tvami
Copy link
Contributor

tvami commented Aug 2, 2022

This is an issue that's marked urgent and I think already resolved, can it be closed?

@mmusich
Copy link
Contributor Author

mmusich commented Aug 2, 2022

It needs to be signed by the respective L2s

@tvami
Copy link
Contributor

tvami commented Aug 2, 2022

@bsunanda you can now sign for both signatures, no?

@civanch
Copy link
Contributor

civanch commented Aug 2, 2022

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2022

This issue is fully signed and ready to be closed.

@mmusich
Copy link
Contributor Author

mmusich commented Aug 2, 2022

closing as resolved in #38864

@mmusich mmusich closed this as completed Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants