-
Notifications
You must be signed in to change notification settings - Fork 3
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
Introduced new class TRestGeant4GeometryInfo #36
Conversation
I think a usage example would be desirable, including a pipeline. |
Would this PR change anything for the user? I guess the pipeline is not changed. May be the example could be just loading an individual gdml file with this new class? |
Yes, this should not change anything for the user. It just adds a new member to The original motivation was that the usage of assembly geometries requires a new way to define a "volume name", since the one provided by Geant4 GDML parser and ROOT differs. This class takes care of this internally, keeping the usual volume names for geometries that are not using assembly. I also added an example to |
New class TRestGeant4GeometryInfo used to store geometry information. Code originally from https://github.com/lobis/radiation-transport, needed to correctly work with assembly gdml geometries.