Fix for RPC ROB based decoding
Passing the vector of hashIDs to decode to the processPad function so that we don't decode the full RPC detector when it isn't needed (this was already done correctly for the PRD based decoding, but for the ROB based decoding the vector being passed was always empty). Because we no longer decode the full RPC detector, this changes slightly the muons we reconstruct in the trigger (and the trigger counts) so the references have been updated accordingly.
Merge request reports
Activity
This merge request affects 3 files:
- MuonSpectrometer/MuonCnv/MuonRPC_CnvTools/src/RpcRdoToPrepDataToolCore.cxx
- Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref
- Trigger/TrigValidation/TriggerTest/share/ref_data_v1Dev_build.ref
Adding @rosati ,@wleight ,@okumura ,@nkoehler ,@jpanduro as watchers
added MuonSpectrometer Trigger master review-pending-level-1 labels
CI Result SUCCESS (hash fdbd8371)Athena AthSimulation AthGeneration AnalysisBase 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
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22816]added changes-trigger-counts review-approved labels and removed review-pending-level-1 label
added 79 commits
-
fdbd8371...434f6fd4 - 78 commits from branch
atlas:master
- ee38d660 - Fix conflicts in reference file
-
fdbd8371...434f6fd4 - 78 commits from branch
This merge request affects 3 files:
- MuonSpectrometer/MuonCnv/MuonRPC_CnvTools/src/RpcRdoToPrepDataToolCore.cxx
- Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref
- Trigger/TrigValidation/TriggerTest/share/ref_data_v1Dev_build.ref
Adding @rosati ,@wleight ,@okumura ,@nkoehler ,@jpanduro as watchers
added review-pending-level-1 label and removed review-approved label
CI Result SUCCESS (hash ee38d660)Athena AthSimulation AthGeneration AnalysisBase 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
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22848]added review-approved label and removed review-pending-level-1 label
mentioned in commit e1efd5c6
added sweep:ignore label