Skip to content

Develop PR: Add new fields to SRTruth/SRTrueInteractions to save meson production time + Assign generator type in CAFMaker - #200

Open
VCLanNguyen wants to merge 4 commits into
developfrom
new_srtrueinteraction_variables
Open

VCLanNguyen wants to merge 4 commits into
developfrom
new_srtrueinteraction_variables

Conversation

@VCLanNguyen

@VCLanNguyen VCLanNguyen commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Description

  1. Adds SRTrueInteraction.prod_time/.dk2gen field
  2. Adds SRMeVPrtl.prod_time
  3. Assigns rec.mc.nu.generator in CAFMaker_module.cc — previously unset

Dependency:

sbncode: SBNSoftware/sbncode#676
sbnanaobj: #200

Validation:

See slide 1-4 on docdb#49588

Quick checklist

  • Have you run git fetch and pulled the latest changes from the branch you're basing your PR against?
  • If you're adding new classes, have you added them to classes_def.xml in the relevant directory?
  • Have you added a checksum in classes_def.xml to any and all new classes you're implementing, and rebuilt?
  • If you're updating classes, have you incremented the ClassVersion by one compared to develop in classes_def.xml?
  • Have you pinged CAF maintainers to let them know of the changes? @Gianluca Petrillo and @thomas Jones

Description

Please provide a short description of your PR.

Adds prod_time (meson decay time from dk2nu ancestry) and dk2gen (decay-to-flux
distance from MCFlux) to caf::SRTrueInteraction, needed to correct the dk2nu
v01_11_00 vertex-time bug for SBND MC.
VCLanNguyen and others added 3 commits September 29, 2026 16:17
Mirrors the dk2nu prod_time field added to SRTrueInteraction: needed
to recover the BNB beam-bunch offset for MeVPrtl (HNL) truth the same
way it's already done for GENIE. Also fixes the stale [us] doc comment
on `time`, which FillMeVPrtlTruth has always filled in ns.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Filled in by checkClassVersion on build, following the prod_time field
addition.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Mirrors SRTrueInteraction's time/prod_time comment style and phrasing
for the analogous SRMeVPrtl fields.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@VCLanNguyen VCLanNguyen changed the title Add dk2nu production-time and baseline-split fields to SRTrueInteraction Develop PR: Add new fields to SRTruth/SRTrueInteractions to save meson production time + Assign generator type in CAFMaker Sep 30, 2026
@VCLanNguyen
VCLanNguyen marked this pull request as ready for review October 1, 2026 18:51
@VCLanNguyen VCLanNguyen self-assigned this Oct 1, 2026
@VCLanNguyen VCLanNguyen added the enhancement New feature or request label Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant