TrigValSteering: Memoize path lookup and add more debug messages
This should hopefully fix ATR-21080. The slowest part of the test configuration, and therefore the unit tests, is the path lookup with find_data
and get_files
. Since they were called multiple times per test, the delays were multiplied. This MR adds the memoize
modifier to these calls to avoid multiple lookups of the same path.
Several extra debug messages are also added to help debug such problems in the future. The time format in the logger will now also show seconds.
Merge request reports
Activity
added Trigger master review-pending-level-1 labels
CI Result SUCCESS (hash 2701ef06)Athena AthSimulation 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 10553]added review-approved label and removed review-pending-level-1 label
mentioned in merge request !30896 (merged)
mentioned in commit 7d24e722
added sweep:ignore label
- Resolved by Rafal Bielski
mentioned in merge request !31073 (merged)