Reorganization of AFP trigger setup

Reorganization of AFP HLT setup and partially offline setup. Now HLT uses the same methods as offline reconstruction. The python setups are placed in respective AFP_Reconstruction packages, not in TriggerMenuMT anymore; the setup of clustering tool is now in AFP_Reconstruction/AFP_SiClusterTools and not part of AFP track reco setup (AFP_Reconstruction/AFP_LocReco). Moreover, further implementation of monitoring tools for trigger should be easier.

Closes ATR-24733

added 1 commit

• 3d96a78e - removing comments

Compare with previous version

marked this merge request as ready

This merge request affects 7 packages:

• ForwardDetectors/AFP/AFP_Reconstruction/AFP_GlobReco
• ForwardDetectors/AFP/AFP_Reconstruction/AFP_LocReco
• ForwardDetectors/AFP/AFP_Reconstruction/AFP_SiClusterTools
• ForwardDetectors/AFP/AFP_Reconstruction/AFP_VertexReco
• ForwardDetectors/ForwardRec
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @goetz ,@sutt ,@hrussell ,@okuprash ,@ggach ,@dzanzi ,@bernius as watchers

added ForwardDetectors Trigger TriggerEDM TriggerMenu TriggerMinBias master review-pending-level-1 labels

CI Result SUCCESS (hash 3d96a78e)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 45067]

Adding user action required label till discussion are resolved.

Feel free to remove it once you are ready.

Cheers, Steffen (L1)

removed review-pending-level-1 label

added review-user-action-required label

added 1 commit

• e4a9fd51 - fix CompFactory commands

Compare with previous version

This merge request affects 7 packages:

• ForwardDetectors/AFP/AFP_Reconstruction/AFP_GlobReco
• ForwardDetectors/AFP/AFP_Reconstruction/AFP_LocReco
• ForwardDetectors/AFP/AFP_Reconstruction/AFP_SiClusterTools
• ForwardDetectors/AFP/AFP_Reconstruction/AFP_VertexReco
• ForwardDetectors/ForwardRec
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @goetz ,@sutt ,@hrussell ,@okuprash ,@ggach ,@dzanzi ,@bernius as watchers

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

CI Result SUCCESS (hash e4a9fd51)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 45101]

Huge changes in the JO. Moving to L2 for deeper look -- L1

added review-pending-level-2 label and removed review-pending-level-1 label

marked this merge request as draft

Removing review-pending-level-2, since the MR has been moved back to draft state.

Jakub (L1 shifter)

removed review-pending-level-2 label