-
Notifications
You must be signed in to change notification settings - Fork 15
Add more NuGraph2 variables to CAFs #183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,16 @@ | ||
| #include "sbnanaobj/StandardRecord/SRNuGraphScore.h" | ||
|
|
||
| namespace caf | ||
| { | ||
|
|
||
| SRNuGraphSliceInfo::SRNuGraphSliceInfo() : | ||
| mip_hits(INT_MIN), | ||
| hip_hits(INT_MIN), | ||
| shr_hits(INT_MIN), | ||
| mhl_hits(INT_MIN), | ||
| dif_hits(INT_MIN), | ||
| ng_vtx_hip_hits(INT_MIN), | ||
| unclustered_shr_hits(INT_MIN) | ||
| { } | ||
|
|
||
| } // end namespace caf |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,9 +5,32 @@ | |
| #define SRNuGraphSCORE_H | ||
|
|
||
| #include "sbnanaobj/StandardRecord/SRConstants.h" | ||
| #include <climits> | ||
|
|
||
| namespace caf { | ||
|
|
||
| /** | ||
| * @brief Information on the slice by NuGraph. | ||
| * | ||
| * This object summarizes the results from running NuGraph over hits in a slice | ||
| * (see e.g. [https://sbn-docdb.fnal.gov/cgi-bin/sso/ShowDocument?docid=40585](SBN DocDB 40585). | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ❤️ |
||
| * | ||
| * The number of HIP hits is computed for a circle around the vertex in each plane. | ||
| */ | ||
| class SRNuGraphSliceInfo { | ||
| public: | ||
|
|
||
| SRNuGraphSliceInfo(); | ||
|
Comment on lines
+20
to
+23
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
|
||
| int mip_hits; ///< Number of hits tagged as `MIP`. | ||
| int hip_hits; ///< Number of hits tagged as `HIP`. | ||
| int shr_hits; ///< Number of hits tagged as `Shower`. | ||
| int mhl_hits; ///< Number of hits tagged as `Michel`. | ||
| int dif_hits; ///< Number of hits tagged as `Diffuse`. | ||
| int ng_vtx_hip_hits; ///< Number of hits tagged as `HIP` around the reconstructed interaction vertex. | ||
| int unclustered_shr_hits; ///< Number of hits tagged as `Shower` not belonging to any reconstructed Pandora PFP object. | ||
| }; | ||
|
|
||
| /** | ||
| * @brief Categorization of the object/PFP by NuGraph. | ||
| * | ||
|
|
@@ -37,6 +60,7 @@ namespace caf { | |
| float dif_frac = caf::kSignalingNaN; ///< Fraction of hits that are labeled as `Diffuse`. | ||
| float bkg_frac = caf::kSignalingNaN; ///< Fraction of hits that are labeled as `Background`. | ||
| }; | ||
|
|
||
| } | ||
|
|
||
| #endif | ||
| Original file line number | Diff line number | Diff line change | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -46,6 +46,13 @@ | |||||||||||||||
| <version ClassVersion="10" checksum="705810055"/> | ||||||||||||||||
| </class> | ||||||||||||||||
|
|
||||||||||||||||
| <class name="caf::SRNuGraphSliceInfo" ClassVersion="15" > | ||||||||||||||||
| <version ClassVersion="15" checksum="2711257956"/> | ||||||||||||||||
| <version ClassVersion="14" checksum="809065913"/> | ||||||||||||||||
| <version ClassVersion="13" checksum="290910659"/> | ||||||||||||||||
| <version ClassVersion="12" checksum="434873519"/> | ||||||||||||||||
|
Comment on lines
+50
to
+53
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove the transient versions from the final commit. |
||||||||||||||||
| </class> | ||||||||||||||||
|
|
||||||||||||||||
| <class name="caf::SRHit" ClassVersion="10"> | ||||||||||||||||
| <version ClassVersion="10" checksum="1494967532"/> | ||||||||||||||||
| </class> | ||||||||||||||||
|
|
@@ -71,7 +78,11 @@ | |||||||||||||||
| <version ClassVersion="10" checksum="83320005"/> | ||||||||||||||||
| </class> | ||||||||||||||||
|
|
||||||||||||||||
| <class name="caf::SRSlice" ClassVersion="22"> | ||||||||||||||||
| <class name="caf::SRSlice" ClassVersion="26"> | ||||||||||||||||
| <version ClassVersion="26" checksum="2614728424"/> | ||||||||||||||||
| <version ClassVersion="25" checksum="3061622568"/> | ||||||||||||||||
| <version ClassVersion="24" checksum="2614728424"/> | ||||||||||||||||
| <version ClassVersion="23" checksum="3732772050"/> | ||||||||||||||||
|
Comment on lines
+81
to
+85
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same.
Suggested change
|
||||||||||||||||
| <version ClassVersion="22" checksum="2124497341"/> | ||||||||||||||||
| <version ClassVersion="21" checksum="3825441135"/> | ||||||||||||||||
| <version ClassVersion="20" checksum="1917004696"/> | ||||||||||||||||
|
|
||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Initialise the variables inline in the class definition rather than using a constructor. This
.cxxwill not be needed any more (or it will be empty).