Feature/truth sign dist mu mom #130
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e further changes + improvements
… add it to the title of each hist
…rectory -- it should be the data area, not app.
…same name as the file...
…ke hists wider, fix legend, and clarify the name option
…ithin the validate function
…f the code a bit. And also puts the sign distance calculation into a function too...
…re finer at low KE and high KE.
…ore -- all in MeV. And also add a vertical line for easier viewing.
… purity calculation..
…small number of events, so we may have integrals of 0, so need to account for this.
… failing bc 0 mm bin was double counted
… the muon KE entering TMS. To do this we: create function to calculate KE from momentum, create a second pair of lists of TH1s for plots of the KE entering TMS. This cause some restructuring to cut down on code -- just make a bigger loop and use zip() to access the pairs of hists -- done for all the visuals and plotting. Clarify the names of some functions, move legend outside of loop, only need one (all are the same), save plots with what det was used to calc the KE. let's also clarify the other list of hists is inside LAr, so add it to the name of the object, and to its titles.
…e code to make plots for Enu slices -- might be better to make new script. Also had issue accessing the index for the neutrino 4-vector, attempt to add plots of the Enu slices as well
…S.D. with slices of Enu.
…ge -- over 5 GeV. skip over them. and address some typos
… lets have a dir for Truth, and also change the outdir so its clear and matches filename
LiamOS
approved these changes
Aug 26, 2024
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the PR for the work on the muon signed distance. Has two scripts:
signed_distance_mu_momentum_slices.py. Create Truth muon (and anti-muon) Signed Distance plots in slices of muon KE. The script creates two versions of the plots:From discussions at Friday meetings, both seem useful, so that's why the script makes both.
signed_distance_enu_momentum_slices.py. Does the same thing, but makes plots of the Signed Distance in slices of the True Neutrino Energy. I have only run this script over a limited number of TMSReco events, so the stats in the plots I have are sparse (also was suggested at the last TMS Studies meeting).I also made a new directory for these scripts,
Truth. I thought they should go there since they use truth info and don't confuse people by putting them in theRecodir.