Add AD HOC generator position for NEXT100#270
Add AD HOC generator position for NEXT100#270kvjmistry wants to merge 1 commit intonext-exp:masterfrom
Conversation
|
This option exists already and it's implemented in NEXT100.cc (and NEXT100OpticalGeometry.cc). In fact, the function you added is never called, because the root geometry is NEXT100 (or NEXT100OpticalGeometry), not NEXT100FieldCage. In case it's not clear, the way this works is that the root geometry propagates the "region" argument to some sub-geometry, which does the same, until one component returns a value. In general, if you add a region identifier for random position generation, you would need to add this propagation all the way to the root geometry (or geometries), but for this specific label, the method is implemented directly in the root geometry because it is not specific to any sub-geometry. |
|
Oops, I have no idea why I didn't check for the root AD_HOC generation. Closing this PR. |
Add AD HOC generator option for NEXT-100 geometry to make it easier to generate events with any specified position in the active volume.