Skip to content
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

p+ contact for the Ortec detectors #50

Open
tdixon97 opened this issue Nov 1, 2024 · 5 comments
Open

p+ contact for the Ortec detectors #50

tdixon97 opened this issue Nov 1, 2024 · 5 comments

Comments

@tdixon97
Copy link
Contributor

tdixon97 commented Nov 1, 2024

Ortec detectors (PPC and some ICPC) are implemented with sunken cylindrical p+ contacts (for example the r,z profile below of P00661C).
image
However, in reality these detectors have a spherical p+ contact geometry.
image

It may be nice to correct this, although I doubt the effect is large.

@ManuelHu
Copy link
Collaborator

ManuelHu commented Nov 1, 2024

I see. I did not originally write this code, but a general problem with the polycones is that they cannot (easily) represent spherical segments.

We could only approximate it by some cone segments...

@tdixon97
Copy link
Contributor Author

tdixon97 commented Nov 2, 2024

I guess you could also make it via a subtraction of a G4Sphere, but the geometry being a subtraction solid not a G4GenericPolycone is annoying and harder to work with

@gipert
Copy link
Member

gipert commented Nov 2, 2024

Yeah no subtractions please. If you think that the square dimple is not ok we could approximate the sphere with a couple more points.

@tdixon97
Copy link
Contributor Author

tdixon97 commented Nov 2, 2024

I think this will only matter if we try to simulate alphas. Maybe we can chose the square depth so as keeping the area correct?

@gipert
Copy link
Member

gipert commented Nov 2, 2024

Yes that's I guess the right thing to do

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

No branches or pull requests

3 participants