Feat: Adding relevant fields to save the rich SPINE output - #91
Feat: Adding relevant fields to save the rich SPINE output#91pgranger23 wants to merge 17 commits into
Conversation
…ndardRecord Added the SRDLPParticle container, flat DLPPid and DLPShape enums, updated SRRecoParticlesBranch to include dlp_detailed vector, and implemented required ROOT dictionary classes registration and proxy generation template instantiations.
…RecoParticle Removed SRDLPParticle and instead added SREnergyEst, SRPIDProbability, SRClusteringInfo, and SRDLPInfo as nested structs inside SRRecoParticle, which also got start_dir, end_dir, and length fields. Checked in auto-generated ROOT dictionary checksums.
…to hypothesis vectors
# Conflicts: # CMakeLists.txt # duneanaobj/StandardRecord/Proxy/Instantiations.cxx # duneanaobj/StandardRecord/SRRecoParticle.h # duneanaobj/StandardRecord/classes_def.xml
…to SRPFP, and group PID-energies
…or of SRRecoParticle directly in SRInteraction
…SRLArRecoParticle
|
Following the comments from our meeting: Summary of Changes since initial submission1. Rename DLP to SPINE
2. Simplify Reconstructed Particles collection (
|
|
I think we're more or less agreed on the fundamentals now, just waiting for signoff from ND. One note: I think you missed an instance of DLP (probably because it's got the full name in it): line 205 of https://github.com/pgranger23/duneanaobj/blob/feat/dlp-detailed/duneanaobj/StandardRecord/SREnums.h#L205 |
|
Good catch, had missed it, thanks! |
Initial proposal on how to include relevant output fields from SPINE into the CAFs.
This is a first guess to start the discussion, I'm open to doing things differently.
Note that many of those fields are genuinely useful and could be populated by other reconstruction algorithms.