Skip to content

Conversation

@kvjmistry
Copy link
Collaborator

This PR adds a generator to make events in a 1 micron thickness, 1 micron in front of the cathode mesh. This is for modelling the plated-out Rn events.

According to the geometry, the cathode mesh sits at z = 1187.37 mm. 100k test events were generated with the following coordinates:

var initial_x initial_y initial_z
min -482.316803 -482.406372 1187.302979
max 482.427917 482.360321 1187.305054

The z positions are just in front of the cathode mesh surface.

image

Copy link
Collaborator

@gonzaponte gonzaponte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I've annotated, the generator is actually 2 um thick, not 1. I don't think this matters, so I approve it anyway. Feel free to change it, if you prefer it to be 1 um.

// Generate in a 1 micron thick disk in front of cathode
cathode_surf_gen_ =
new CylinderPointSampler(0.0, cathode_int_diam_/2.,
1.0*micrometer,0., twopi, nullptr,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not that it matters, but to get 1 um, you should specify 0.5*micrometer here.

Copy link
Collaborator Author

@kvjmistry kvjmistry Jun 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah yes, the half-width!
2 micron is fine shouldn't make much of a difference. I'll update the thickness so it matches with the desired shift.

1.0*micrometer,0., twopi, nullptr,
G4ThreeVector(GetCoordOrigin().x(),
GetCoordOrigin().y(),
cathode_grid_zpos - grid_thickn_ - 1*micrometer));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here.

2 um is fine, though :)

@kvjmistry kvjmistry merged commit 2a66f9d into next-exp:master Jun 17, 2025
1 check passed
@kvjmistry kvjmistry deleted the CathSurfGen 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