Skip to content

Conversation

@kvjmistry
Copy link
Collaborator

Add AD HOC generator option for NEXT-100 geometry to make it easier to generate events with any specified position in the active volume.

@gonzaponte
Copy link
Collaborator

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.

@kvjmistry
Copy link
Collaborator Author

Oops, I have no idea why I didn't check for the root AD_HOC generation. Closing this PR.

@kvjmistry kvjmistry closed this Nov 4, 2025
@kvjmistry kvjmistry deleted the AD_HOC_Gen branch December 5, 2025 16:22
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