Update Pandora scripts to use the new outerface for track/shower PID#75
Merged
Update Pandora scripts to use the new outerface for track/shower PID#75
Conversation
Member
|
Not a formal review, I haven't tested this, but just to say thanks @jback08 for the work and the level of detail in the description. At a high level the changes and additions look good to me, and I am looking forward to seeing this running. |
jacoblarkin
approved these changes
Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated the
run-pandorascripts to use the new outerface for storing track/shower PID info in the Pandora ND CAFs (ND_CAFMaker PR110) as well as using rock-muon cheating for ND-LAr production (to significantly speed up the Pandora reco for full-spill events):setup_pandora.shuses the latest Pandora package versions, as well as introducing new environment variables to control the running of the Pandora reco & outerface steps (xml settings files & reco options).install_pandora.shupdates the machine learning data download options, which gets the files using thedownload.shscript from Pandora'sLArMachineLearningDatapackage. The files in/global/cfs/cdirs/dune/www/data/pandora/LArMachineLearningData/will need to be replaced with the new ones.run_pandora.shandrun_pandora.data.shnow run the extraPandoraOuterfacestep, which uses as input the hierarchy analysis ROOT file fromPandoraInterface. The analysis outputLArRecoND.rootis renamedLArRecoNDHierarchy.rootfor thePandoraOuterface, which then creates the final Pandora reco output fileLArRecoND.root. This means that we don't need to change theLArRecoND.rootfilename used for the CAF-making scripts.isFinal(0 = prompt, 1 = final, 2 = merged) &legacyMode(default = 0)The ND-LAr production should use the run settings file
LArRecoND/settings/PandoraSettings_LArRecoND_ThreeD_PartialCheated.xmlto cheat the rock muons but fully reconstruct the neutrino interactions. This xml file is now specified by theND_PRODUCTION_PANDORA_LAR_RECO_ND_XMLenvironment variable.The 2x2 production can still use the run settings file
LArRecoND/settings/PandoraSettings_LArRecoND_ThreeD.xml, again set byND_PRODUCTION_PANDORA_LAR_RECO_ND_XML.Pandora ND-LAr reco requires either prompt or merged flow hits (
isFinal= 0 or 2, respectively)