adding missing et conversion factor for KFMET definition for Ph1Topo
This MR updates the final cut values/thresholds for the Ph1Topo based KFMET algorithm to more realistic values. The algorithm itself is still under commissioning and will certainly require more updates (thresholds but also filter coefficients) once technically commissioned. Also the L1Topo simulation for this algorithm will still need updates. Yet, having more realistic thresholds configured in hardware at P1 is desirable to ease the commissioning of the firmware implementation, especially once LHC provides high energy+rate collisions again.
Related to https://its.cern.ch/jira/browse/ATR-28657
Merge request reports
Activity
added 24.0 Trigger TriggerMenu review-pending-level-1 labels
added Run3-MCReco-output-changed label
CI Result FAILURE (hash a8caff46)Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6917] (remote access info)This merge request affects 10 packages:
- Event/xAOD/xAODForwardCnv
- Event/xAOD/xAODMuonCnv
- Event/xAOD/xAODTauCnv
- Event/xAOD/xAODTrigCaloCnv
- Event/xAOD/xAODTrigMinBiasCnv
- Event/xAOD/xAODTrigMissingETCnv
- Event/xAOD/xAODTriggerCnv
- MuonSpectrometer/MuonCnv/MuonNSWCommonDecode
- Trigger/TrigSteer/DecisionHandling
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @apsallid ,@stavrop ,@adbailey ,@kluit ,@lidiaz ,@akraszna ,@xiaozhon ,@tamartin ,@pscholer ,@damazio ,@martindl ,@rosati ,@jojungge ,@stelzer ,@sutt as watchers
added EDM MuonSpectrometer Tau analysis-review-required labels
- Resolved by Frank Winklmeier
The Robot seems a bit too eager to identify "affected" packages and applying labels... Also the pre-rebase pipeline seemed to fail for unrelated reasons.
removed MuonSpectrometer label
CI Result FAILURE (hash b9882da9)Athena DetCommon externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6924] (remote access info)mentioned in merge request !69714 (merged)
This merge request affects 10 packages:
- Event/xAOD/xAODForwardCnv
- Event/xAOD/xAODMuonCnv
- Event/xAOD/xAODTauCnv
- Event/xAOD/xAODTrigCaloCnv
- Event/xAOD/xAODTrigMinBiasCnv
- Event/xAOD/xAODTrigMissingETCnv
- Event/xAOD/xAODTriggerCnv
- MuonSpectrometer/MuonCnv/MuonNSWCommonDecode
- Trigger/TrigSteer/DecisionHandling
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @kluit ,@stavrop ,@sutt ,@stelzer ,@jojungge ,@adbailey ,@pscholer ,@rosati ,@xiaozhon ,@martindl ,@akraszna ,@tamartin ,@damazio ,@apsallid ,@lidiaz as watchers
added MuonSpectrometer label
- Resolved by Frank Winklmeier
There is something very strange going on with your branch. How did you rebase it?
CI Result FAILURE (hash b9882da9)Athena DetCommon externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6949] (remote access info)removed EDM label
removed MuonSpectrometer label
removed Tau label
removed Run3-MCReco-output-changed label
removed review-pending-level-1 label
added review-approved label
removed analysis-review-required label
added analysis-review-approved label
added review-approved-point1 label
mentioned in commit 83aff564
mentioned in merge request !70129 (merged)