-
Notifications
You must be signed in to change notification settings - Fork 5
Description
Right now dune-tms needs to know certain information about the geometry. It gets it through the root geometry provided by edep-sim. We can get some info through FindNode. But FindVolumeFast seems to be broken. That makes it difficult to query specific sizes of things for example
Beyond that, some numbers aren't in the geometry right now. Like we care about TMS active volume. This is defined as the cuboid that contains all overlapping scintillator
We currently use a config file to define it, but that can go very wrong if the config used is mismatched with the input file. Worse, for the new double thick geom we have a whole different branch. That's because some values are hard coded
See this issue for possible ways of doing it, plus what values dune-tms cares about:
DUNE/dune-tms#204
Whatever choice we use, it needs to survive the mangling of root and edep-sim. It doesn't help to get something in the gdml if it's not loaded into the root geometry. We may be able to fix edep-sim but that would be more long term. We'd need to submit a PR and then get a new build, and that can take a while in my experience