Implementing muoncalib chains in Run-3 menu, ATR-20650

I want to merge this change into master.

I implemented muoncalib chains in the Run-3 menu.

• HLT_mu0_muoncalib_L1MU4_EMPTY for cosmic menu
• HLT_mu0_muoncalib_L1MU20 for detector monitoring

I added an option to change the settings for these muoncalib chains to TrigHypothesis/TrigMuonHypoMT/TrigMufastHypoTool.cxx. This option only monitors variables (same as TrigHypothesis/TrigMuonHypo/TrigMufastCALHypo.cxx in Run-2 code). I also added these monitoring histograms in expert-monitoring.root.

tagging @nakahama and @sshaw for info.

This merge request affects 2 packages:

• Trigger/TrigHypothesis/TrigMuonHypoMT
• Trigger/TriggerCommon/TriggerMenuMT

added Trigger TriggerMenu master review-pending-level-1 labels

CI Result FAILURE (hash 4c40134e)

	Athena	AthSimulation	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
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11100]

Please resolve all open threads, then re-add the review-pending-level-1 label. Thanks!

MLB (L1)

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• 7557d51b - Fix MufastHypoTool and monitoring, and update refarence

Compare with previous version

This merge request affects 4 packages:

• Trigger/TrigHypothesis/TrigMuonHypoMT
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TrigUpgradeTest
• Trigger/TriggerCommon/TriggerMenuMT

Adding @hartj ,@jpanduro as watchers

added review-pending-level-1 label and removed review-user-action-required label

CI Result FAILURE (hash 7557d51b)

	Athena	AthSimulation	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
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11153]

Hi Moe,

you will need to update the references for the tests trigAna_RDOtoRDOTrig_mt1_build and test_trig_data_v1Dev_build.py since you've added new chains (you can run the tests locally and follow the instructions for updating the references). The other test failures I think are unrelated to your changes.

Cheers,

Savanna

added review-user-action-required label and removed review-pending-level-1 label

added 71 commits

• 7557d51b...2f5108aa - 69 commits from branch atlas:master
• abbbc564 - Merge remote-tracking branch 'upstream/master' into Implement_muoncalib
• 4b791e2a - Monitored all valiables and updated reference

Compare with previous version

This merge request affects 5 packages:

• Trigger/TrigHypothesis/TrigMuonHypoMT
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TrigUpgradeTest
• Trigger/TrigValidation/TriggerTest
• Trigger/TriggerCommon/TriggerMenuMT

Adding @hartj ,@jpanduro as watchers

added review-pending-level-1 label and removed review-user-action-required label

resolved all threads

CI Result SUCCESS (hash 4b791e2a)

	Athena	AthSimulation	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11171]

Looks good, warning is unrelated, approving (L1)