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

flux generation radii #83

Closed
tlatorre-uchicago opened this issue Jan 7, 2020 · 3 comments
Closed

flux generation radii #83

tlatorre-uchicago opened this issue Jan 7, 2020 · 3 comments
Assignees

Comments

@tlatorre-uchicago
Copy link
Contributor

tlatorre-uchicago commented Jan 7, 2020

A small issue I came across when using the gevgen_atmo script is that by default it sets the radii for the flux generation surfaces to be be 1 here:

atmo_flux_driver->SetRadii(1, 1);
. However if the actual geometry is much bigger than 1 then this will incorrectly sample the interaction vertices.

@candreop
Copy link
Member

candreop commented Jan 7, 2020

gevgen_atmo will need a small addition, providing command line options to specify the radii - thanks for bringing this up. Will be fixed asap

@candreop candreop self-assigned this Jan 7, 2020
@tlatorre-uchicago
Copy link
Contributor Author

tlatorre-uchicago commented Jan 14, 2020

I might be able to work on this if you could provide me with a small amount of help. Do you know if it's possible to directly get the overall bounding box for the specified geometry? If so, do you think it would be reasonable to use a sphere big enough to encompass the bounding box of the geometry?

The other question I had was why there are two radii. I looked into the GAtmoFlux class for a bit but couldn't determine what it is. Is the flux generation surface an ellipsoid?

candreop added a commit that referenced this issue Jan 15, 2020
… allow users to set the distance and radius of the flux ray generation surface
@candreop
Copy link
Member

candreop commented Jan 15, 2020

Sorry, was away at a workshop. Added 2 new command line arguments in gevgen_atmo (in the master). They are: --flux-ray-generation-surface-distance and --flux-ray-generation-surface-radius. They allow you to set the parameters of the flux ray generation surface. No the surface is not an ellipsoid. For a given zenith \theta and azimuthal angle \phi, flux rays are produced within the area of a circle of radius Rt, which is tangetial to a point P on a sphere of radius Rl, centred at the detector. The point P has polar coordinates \theta and \phi. You specify Rt and Rl. Th eatteched plot might help understanding the setup
Screenshot 2020-01-08 at 05 43 45

If you have some coding skills and want to contribute, I would like to invite you to help us with improving the efficiency of gevgen_atmo. It uses a full-blown geometry navigation driver (that we use for more complex simulation cases with accelerator based neutrinos close to the source) but this is an overkill for atmospheric neutrinos since all parts of the detector see the same beam. Some simplifications could make gengen_atmo run much faster.

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

2 participants