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

Adding weight branches necessary for uB comparison #342

Open
wants to merge 1 commit into
base: Application
Choose a base branch
from

Conversation

jminock
Copy link
Contributor

@jminock jminock commented Feb 19, 2025

Describe your changes

Adds weight branches to PhaseIITreeMaker. These branches are necessary to reflect all of MicroBooNE's XSec reweights for the joint analysis. These branches fit alongside existing weight branches and follows their conventions. Branch "weight_All_UBGenie" is removed because keeping it would create handling conflicts in STV-analysis (downstream of ToolAnalysis) code for joint cross-section plots.

I also removed a couple forward slashes from a comment block in PhaseIITreeMaker. Doing so fixes a visual bug in vim below the comment block where everything was highlighted as a comment. This change does not uncomment anything, add any additional comments, or change the resultant PhaseIITree ntuple.

Checklist before submitting your PR

  • This PR implements a single change (one new/modified Tool, or a set of changes to implement one new/modified feature)
  • This PR alters the minimum number of files to affect this change
  • [ NA ] If this PR includes a new Tool, a README and minimal demonstration ToolChain is provided
  • [ NA ] If a new Tool/ToolChain requires model or configuration files, their paths are not hard-coded, and means of generating those files is described in the readme, with examples provided on /pnfs/annie/persistent
  • For every new usage, there is a reason the data must be on the heap
  • For every new there is a delete, unless I explicitly know why (e.g. ROOT or a BoostStore takes ownership)

Additional Material

NA

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.

1 participant