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

CreateCaloCellPositions with hard coded system ID #430

Open
BrieucF opened this issue Dec 2, 2020 · 0 comments
Open

CreateCaloCellPositions with hard coded system ID #430

BrieucF opened this issue Dec 2, 2020 · 0 comments

Comments

@BrieucF
Copy link
Contributor

BrieucF commented Dec 2, 2020

CreateCaloCellPositions currently has hard coded system ID (link) which breaks the algorithm if system ID changes in the detector description xml. I tried to retrieve the various system ID based on the geometry but could only have it work with try/catch handling because there are subsystems in FCC-hh that are not present in FCC-ee: link. Maybe the best would be to have this tool running on one subsystem at a time to remove the subsystem ambiguity, and call it once for each subsystem in the python cfg?

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

1 participant