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

Knowledge of muon momentum is assumed during X0 inference #41

Open
GilesStrong opened this issue Jun 2, 2021 · 1 comment
Open

Knowledge of muon momentum is assumed during X0 inference #41

GilesStrong opened this issue Jun 2, 2021 · 1 comment
Labels
enhancement New feature or request Inference Issue affects the quality of inference medium priority Should be fixed soon, but doesn't disastrously impact project Realism Issue affects what would actually be possible in application

Comments

@GilesStrong
Copy link
Owner

Current status

X0Inferer.x0_from_dtheta requires MuonBatch.reco_mom in order to compute X0 during the scattering inversion, however MuonBatch.reco_mom simply returns the gen.-level momentum MuonBatch.mom. In actual application, the momentum of the muons must be estimated.

Possible solution

According to @giamman (please correct if wrong), this is usually achieved by placing material of known X0 below the detector under the passive volume, and then adding a third detector layer; scattering in the second passive volume can then be used to invert the scattering formula in terms of momentum, and then the reco momentum can be used to estimate the X0 in the first passive volume.

This can be achieved in code by slightly generalising ScatterBatch to use user-specified hit locationsand then using a class similar toX0Infererto infer the momentum, and update the muon-batch properties prior to running the X0 inference. Generalisation and checking of theVolumeandVolumeWrapperclasses will be required to account for the two passive volumes. Additionally, the second passive volume could be constructed from a newLayer` class, and the material used could be an optimisable parameter.

@GilesStrong GilesStrong added enhancement New feature or request good first issue Good for newcomers medium priority Should be fixed soon, but doesn't disastrously impact project labels Jun 2, 2021
@giamman
Copy link
Collaborator

giamman commented Jun 2, 2021 via email

@GilesStrong GilesStrong added Inference Issue affects the quality of inference Realism Issue affects what would actually be possible in application labels Jun 4, 2021
@GilesStrong GilesStrong removed the good first issue Good for newcomers label Jun 17, 2021
@GilesStrong GilesStrong added this to Nice to have in VoxelNet Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Inference Issue affects the quality of inference medium priority Should be fixed soon, but doesn't disastrously impact project Realism Issue affects what would actually be possible in application
Projects
VoxelNet
Nice to have
Development

No branches or pull requests

2 participants