Skip to content

Blip True EDep from sim:IDE Object#858

Merged
nathanielerowe merged 9 commits intodevelopfrom
feature/blip_changeto_simide
Nov 19, 2025
Merged

Blip True EDep from sim:IDE Object#858
nathanielerowe merged 9 commits intodevelopfrom
feature/blip_changeto_simide

Conversation

@Jjm321814
Copy link
Copy Markdown
Contributor

Blips got their MC truth information from sim::EnergyDeposit objects which are dropped in standard sbnd processing. This patch changes the blip algorithm to dig through sim::Channels and extract the relevant IDE vectors

@Jjm321814
Copy link
Copy Markdown
Contributor Author

I also updated blip reco to provide edep information for gamma and neutrons
image

Running
lar -c standard_reco2_sbnd.fcl -s /pnfs/sbn/data_add/sbn_nd/poms_production/mc/MCP2025_FallValidation/v10_10_03_02/prodcorsika_proton_intime_sbnd/CV/gen_g4_detsim_reco1/df/28/gen_g4_detsim_reco1-2549e99c-9c70-4a5f-9ead-375f8175ce72.root
I see the EDep object is filled
image

And finally running over a e- gun sample from 0 MeV to 2 MeV we see EDep values equal to the e- kinetic energy
image

@Jjm321814 Jjm321814 self-assigned this Oct 24, 2025
@nathanielerowe
Copy link
Copy Markdown
Contributor

@sungbinoh Can you take a look at this?

Copy link
Copy Markdown
Contributor

@sungbinoh sungbinoh left a comment

Choose a reason for hiding this comment

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

Hi @Jjm321814 , thank you for the updates.
It looks good for me. I have only two minor suggestions.

Comment thread sbndcode/BlipRecoSBND/Utils/BlipUtils.cc
Comment thread sbndcode/BlipRecoSBND/Alg/BlipRecoAlg.cc Outdated
Copy link
Copy Markdown
Contributor

@sungbinoh sungbinoh left a comment

Choose a reason for hiding this comment

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

Thank you for the updates! It looks good for me.

@Jjm321814
Copy link
Copy Markdown
Contributor Author

Merged in changes to the producer module to also save collection plane hit clusters. More details are in the (now closed) PR 862

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure if this line works... We will see if CI complains.

@linyan-w linyan-w moved this to Reviewer Approved in SBND 2025 Fall Production Nov 10, 2025
@nathanielerowe
Copy link
Copy Markdown
Contributor

trigger build LArSoft/lar*@LARSOFT_SUITE_v10_11_01

@FNALbuild
Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

❌ CI build for SBND Failed at phase build SBND on slf7 for c14:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the build SBND phase logs

parent CI build details are available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

❌ CI build for SBND Failed at phase build SBND on slf7 for e26:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the build SBND phase logs

parent CI build details are available through the CI dashboard

@nathanielerowe
Copy link
Copy Markdown
Contributor

trigger build LArSoft/lar*@LARSOFT_SUITE_v10_12_02 SBNSoftware/sbndaq-artdaq-core@v1_10_06 SBNSoftware/sbn*@SBN_SUITE_v10_12_02

@FNALbuild
Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

❌ CI build for SBND Failed at phase build SBND on slf7 for c14:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the build SBND phase logs

parent CI build details are available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

⚠️ CI build for SBND Warning at phase ci_tests SBND on slf7 for e26:prof -- details available through the CI dashboard

🚨 For more details about the warning phase, check the ci_tests SBND phase logs

parent CI build details are available through the CI dashboard

@nathanielerowe
Copy link
Copy Markdown
Contributor

Approved

@nathanielerowe nathanielerowe moved this from Reviewer Approved to RM Approved in SBND 2025 Fall Production Nov 18, 2025
@nathanielerowe nathanielerowe merged commit 44c55d9 into develop Nov 19, 2025
@nathanielerowe nathanielerowe moved this from RM Approved to In tagged release in SBND 2025 Fall Production Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In tagged release

Development

Successfully merging this pull request may close these issues.

5 participants