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

[IDEA] Defining surface plugin for IDEA muon-system + modifying the hierarchy of the detElement to cope with surfaces #436

Merged
merged 4 commits into from
Feb 23, 2025

Conversation

mahmoudali2
Copy link
Member

BEGINRELEASENOTES

  • Adding general surface plugin for IDEA muon-system.
  • Fixing of the hierarchy of the detElemnt (the former one had errors when running with surfaces).
  • Now we have ~ 7256 surfaces, but 'ddPlanarDigi' still complaining that it doesn't see surfaces!

ENDRELEASENOTES

@atolosadelgado
Copy link
Collaborator

Hi @mahmoudali2

Thanks for this PR!

Could you please address the CI test that is failing? link

This is the error message:

IDDescriptor     ERROR dd4hep: PreshowerSystemCollection: This ID descriptor has no field with the name: side
DD4hep_VolumeManager ERROR Exception: IDDescriptor: dd4hep: PreshowerSystemCollection: This ID descriptor has no field with the name: side
     dd4hep: while programming VolumeManager. Are your volIDs correct?

@mahmoudali2
Copy link
Member Author

Sorry @atolosadelgado I forgot to adopt pre-shower with the updates on the builder, but now it's done.. I updates the pre-shower xml too.

@atolosadelgado
Copy link
Collaborator

Hi @mahmoudali2

thanks! The code seems reasonable to me. Do you think it is worth to update the readme with your changes? link

@mahmoudali2
Copy link
Member Author

Done. @atolosadelgado

@atolosadelgado atolosadelgado merged commit 6344ff0 into key4hep:main Feb 23, 2025
7 checks passed
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

Successfully merging this pull request may close these issues.

2 participants