Skip to content

Conversation

@seokjuchung
Copy link

@seokjuchung seokjuchung commented Jan 30, 2026

Description

Please provide a detailed description of the changes this pull request introduces. If available, also link to a docdb link where the issue/change have been presented on/discussed.

The previous prodgenie_nu_singleinteraction_tpc_NCDeltaRadiative_filtered_sbnd.fcl did not pass any events. This has been fixed by using the working prodgenie_nu_singleinteraction_tpc_NCRadiativeResonant_filtered_sbnd.fcl structure and editing the Resonant filter to the NCDeltaRadiative filter.

This working version has a passing rate of 0.5%.

  • Have you added a label? (bug/enhancement/physics etc.)
  • Have you assigned at least 1 reviewer?
  • Is this PR related to an open issue / project?
    This PR is related not related to an open issue / project
  • Does this PR affect CAF data format? If so, please assign a CAF maintainer as additional reviewer.
    No CAF format change
  • Does this PR require merging another PR in a different repository (such as sbnanobj/sbnobj etc.)? If so, please link it in the description.
    Does not require different PR
  • Are you submitting this PR on behalf of someone else who made the code changes? If so, please mention them in the description.

# modules
#

#include "filters_sbnd.fcl"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What filters is this removing?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sbnd_finalstateparticlefilter, sbnd_ccnuefilter, and standard_crt_trig_filter. See https://github.com/SBNSoftware/sbndcode/blob/develop/sbndcode/Filters/fcls/filters_sbnd.fcl

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this could well have been the cause I reckon, if a charged lepton is required in the FS and you have an NC interaction...

@@ -108,5 +102,5 @@ outputs:
# override
# THIS DOES NOT WORK, CHECK!
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just being careful -- I'm guessing this is an obsolete comment? (i.e. the 0.5% pass rate is indeed all NC-RES events, they are in the TPC, etc..)?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an obsolete comment. Note that while labeled as NCRES the actual filter being used in line 62 is for NCDeltaRAD

@kjplows kjplows assigned seokjuchung and unassigned kjplows Feb 2, 2026
@kjplows kjplows added the bugfix Addresses one or more bugs label Feb 2, 2026
@kjplows kjplows moved this to Open pull requests in SBN software development Feb 2, 2026
@kjplows
Copy link
Contributor

kjplows commented Feb 2, 2026

Looks good to me, I'll ping @miquelnebot from the Sim side to check this is all good! Thanks @seokjuchung

@kjplows kjplows requested a review from miquelnebot February 2, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Addresses one or more bugs

Projects

Status: Open pull requests

Development

Successfully merging this pull request may close these issues.

3 participants