Muon trigger SF update
The previous MR ( !73427 (merged)) for the update of Muon Trigger SF files missed an important part of the update on the run numbers for the different years.
Merge request reports
Activity
This merge request affects 1 package:
- PhysicsAnalysis/MuonID/MuonIDAnalysis/MuonEfficiencyCorrections
This merge request affects 1 file:
- PhysicsAnalysis/MuonID/MuonIDAnalysis/MuonEfficiencyCorrections/Root/MuonTriggerScaleFactors.cxx
Adding @jojungge ,@sangelid ,@apsallid ,@vaustrup ,@nakahama ,@nicolaid as watchers
added Analysis Trigger analysis-review-required main review-pending-level-1 labels
added Run3-DataReco-output-changed label
❌ CI Result FAILURE (hash 07ffbf98)Athena AnalysisBase AthAnalysis 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
✅ AnalysisBase: number of compilation errors 0, warnings 0
✅ AthAnalysis: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 14014] (remote access info)- Resolved by Tadej Novak
added review-user-action-required label and removed review-pending-level-1 label
removed Run3-DataReco-output-changed label
removed review-user-action-required label
added review-pending-level-1 label
This merge request affects 1 package:
- PhysicsAnalysis/MuonID/MuonIDAnalysis/MuonEfficiencyCorrections
This merge request affects 1 file:
- PhysicsAnalysis/MuonID/MuonIDAnalysis/MuonEfficiencyCorrections/Root/MuonTriggerScaleFactors.cxx
Adding @sangelid ,@nakahama ,@nicolaid ,@jojungge ,@vaustrup ,@apsallid as watchers
✅ CI Result SUCCESS (hash 07ffbf98)Athena AnalysisBase AthAnalysis 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
✅ AnalysisBase: number of compilation errors 0, warnings 0
✅ AthAnalysis: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 14038] (remote access info)removed analysis-review-required label
added analysis-review-approved label
mentioned in merge request !73427 (merged)
This merge request affects 1 package:
- PhysicsAnalysis/MuonID/MuonIDAnalysis/MuonEfficiencyCorrections
This merge request affects 1 file:
- PhysicsAnalysis/MuonID/MuonIDAnalysis/MuonEfficiencyCorrections/Root/MuonTriggerScaleFactors.cxx
Adding @sangelid ,@vaustrup ,@jojungge ,@nicolaid ,@apsallid ,@nakahama as watchers
✅ CI Result SUCCESS (hash 0fbcd52b)Athena AnalysisBase AthAnalysis 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
✅ AnalysisBase: number of compilation errors 0, warnings 0
✅ AthAnalysis: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 14067] (remote access info)- Resolved by Michael Holzbock
- Resolved by Michael Holzbock
removed review-pending-level-1 label
added review-user-action-required label
- Resolved by Michael Holzbock
Hmm, there might be a good chance that Federica is on vacation currently; it would be fantastic to have this MR in the release to be built this week so we can finally use the muon trigger SFs for 2023. As the open threads are minor and only suggestions for small code improvements: any objections if we close them for now and move on?
removed review-user-action-required label
added review-approved label
mentioned in commit aa717c4d