Debugging changes to (MuonSegment)PseudoJetAlgorithm
Addresses issue noted on ATLJETMET-1248, where the debug message attempts to access a pointer that has been moved.
@rbielski I tested with q431 on DEBUG -- this hit an error, no segfault
Expanded: To fix the failing test I found myself needing to make a bunch more changes to fix some dodgy configuration in JetRecAlgTestCfg.py and took advantage of this to clean up the ghost configuration, which had some redundancies introduced. Notably:
- GhostScale was hardcoded in at least one place, so I just eliminated this property entirely. I generally dislike string parsing, but in this case I thought the best thing was to just check for "Ghost" in the label, rather than have people write a non-ghost label and then have the C++ prepend "Ghost" (which would also cause problems if we want e.g. HLT to have distinct container names).
- I added a check in PseudoJetMerger so that it should not accept a list of PJCs if there is not exactly one non-ghost container (though I don't require this to be the first one, which is something we could consider enforcing). I'd actually even prefer that we explicitly configure the constituents separately from the ghosts, but we can discuss this.
I also switched JetRecAlgTestCfg to use the new JetTrimming class and made sure this was actually doing something useful (the previous setup with JetTrimmer just did nothing every event because it had no JetPseudojetRetriever). This seems to work...
Merge request reports
Activity
added JetEtmiss Reconstruction master review-pending-level-1 labels
CI Result FAILURE (hash 015a1e89) Athena AthSimulation AnalysisBase AthGeneration externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15244] added review-user-action-required label and removed review-pending-level-1 label
added review-pending-level-1 label and removed review-user-action-required label
added review-user-action-required label and removed review-pending-level-1 label
CI Result FAILURE (hash d27841b9) Athena AthSimulation AnalysisBase AthGeneration externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15282] added 1 commit
- 1707fc83 - Fix clients incl trigger for removed GhostScale property
This merge request affects 3 packages:
- Reconstruction/Jet/JetRec
- Reconstruction/Jet/JetRecConfig
- Trigger/TriggerCommon/TriggerMenuMT
Adding @goetz as watcher
added Trigger TriggerMenu review-pending-level-1 labels and removed review-user-action-required label
CI Result SUCCESS (hash 1707fc83) Athena AthSimulation AnalysisBase AthGeneration externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15294] mentioned in merge request !33615 (closed)
added review-approved label and removed review-pending-level-1 label
mentioned in commit 1bd19be4
added sweep:ignore label