-
Notifications
You must be signed in to change notification settings - Fork 101
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
ddsim: support non-cylindrical tracking region #1302
Comments
I think one would need to implement a Geant4TVUserParticleHandler, where TV is for tracking_volume? (Or TR for tracking region?). Maybe I missed the use of the tracking_volume? |
Aaah ok, the TC in Geant4TCUserParticleHandler stands for tracking cylinder... so there is no way yet to use this feature? |
As far as I saw there isn't one yet. |
In principle that looks simple. I just don't know how I would retrieve that volume in |
nvm. I think I found a way :) I will prepare a PR |
I would like to have a non-cylindrical tracking region as introduced in #384, particularly because in the naive cylindrical approach the lumi cal of my detector is inside the tracking volume.
When I add such a tracking region to my detector and remove the previous definition of the tracking region:
ddsim stops here:
DD4hep/DDG4/python/DDSim/Helper/ParticleHandler.py
Lines 129 to 139 in 7967bdf
(If I don't remove it the cylinder is still used as the tracking region as far as I can tell.)
The
TrackingRegion
example in theexamples/ClientTests
seems to not use ddsim:https://github.com/AIDASoft/DD4hep/blob/master/examples/ClientTests/scripts/TrackingRegion.py
The text was updated successfully, but these errors were encountered: