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

Try putting tutorial code into your package #1

Open
wants to merge 96 commits into
base: main
Choose a base branch
from

Conversation

aolivier23
Copy link

Make sure you commit everything before attempting this. I deleted your old file structure and replaced it with mine. So I don't know what will happen. Hopefully, git leaves your stuff alone and just adds my fiies/directories.

aolivier23 and others added 23 commits April 20, 2021 14:12
… Got rid of directories I don't need and moved everything up to main directory.
…the current directory. So, I can decide where I want plots to end up more easily by running runEventLoop from that directory.
…h is good enough. Numerator has some weird several percent problem.
…n's 2D efficiency. Added some playlists for processing MasterAnaDev tuples. Will work on a better system to change playlists later this week.
…r handling. Compiles, but I haven't tried to run it yet.
… it at least suggests the right files. I had the "complete" command backwards.
README.md installation section now points to the wiki as the first source of installation instructions.  The old installation instructions made some assumptions about installation prefixes that other people don't think of.  The new instructions make it very clear how my installation directories are set up throughout the build process.
@sultamehr sultamehr self-requested a review May 20, 2021 16:36
dlast44 and others added 29 commits June 4, 2021 11:58
…tematics were skipped. Needs at least one flux histogram in the MC reco only.
…les to the unfolded result instead of MC selected. Removed some comments that are no longer needed.
…he closure test without systematics. I have to use at least 2 universes in all bands to avoid triggering "spread errors".
…is prevents a crash that was stopping the cross section .root file from being written.
…uon momentum reweight for these antineutrino playlists. An equivalent bug in MAT-MINERvA only had a minimal impact on my antineutrino thesis analysis.
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.

3 participants