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

PandoraPFA for IDEA-LAr #18

Open
kjvbrt opened this issue Mar 3, 2022 · 3 comments
Open

PandoraPFA for IDEA-LAr #18

kjvbrt opened this issue Mar 3, 2022 · 3 comments

Comments

@kjvbrt
Copy link
Contributor

kjvbrt commented Mar 3, 2022

Attempting to run PandoraPFA inside DDMarlinPandora inside k4MarlinWrapper with the following steering instructions:

from Configurables import MarlinProcessorWrapper

pandora = MarlinProcessorWrapper('DDMarlinPandora')
pandora.OutputLevel = DEBUG
pandora.ProcessorType = 'DDPandoraPFANewProcessor'
pandora.Parameters = {
    'Verbosity': ['WARNING'],
    'PandoraSettingsXmlFile': ['/some/path'],
    'CreateGaps': [False],
    'ECalCaloHitCollections': ['ECalBarrelCells']
}
ApplicationMgr().TopAlg += [pandora]

results in the following error:

[ ERROR "DDMarlinPandora"] Failed to initialize marlin pandora: std exception  getExtension: selection is not unique (or empty)  includeFlag: DetType( 0x812) : CALORIMETER, BARREL, ELECTROMAGNETIC,  excludeFlag: DetType( 0x20020) : FORWARD, AUXILIARY,  --- found detectors : 

Full steering macro is in this gist and output log are in the logfile.txt.

@kjvbrt
Copy link
Contributor Author

kjvbrt commented Mar 3, 2022

@fdplacido's suggestion to add type_flags like here:
https://github.com/kjvbrt/FCCDetectors/tree/det_type
does not seems to change anything.

@andresailer
Copy link
Contributor

Hi @kjvbrt
Is the error message really still the same?

The next step is adding the extension that is expected
https://github.com/iLCSoft/lcgeo/blob/4a5b6bb293ce5526ed23d14a3b5157a31326caf8/detector/calorimeter/GenericCalBarrel_o1_v01_geo.cpp#L94-L98

and a lot more in the rest of that file.

@andresailer
Copy link
Contributor

To use the detector_type flags, this XML file to define the flag IDs has to be included like this:

https://github.com/iLCSoft/lcgeo/blob/cb87609446255c3a94da867ad7801a62ff3b6b05/FCCee/compact/FCCee_o1_v04/FCCee_o1_v04.xml#L275

    <include ref="${DD4hepINSTALL}/DDDetectors/compact/detector_types.xml"/>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants