Record data produced by the fast HLT verterx z-finder algorithm

The MR is a small step forward in getting a pileup robust HMT trigger. It adds a recording of the z-vertex data in the HLT EDM (very small collection). Tagging @algilber So far it is a draft because I could not test it properly w/o a full nightly build.

marked this merge request as ready

added 1 commit

• 7e7cd65a - Declare decorations, python config issues fixed

Compare with previous version

Also tagging @somadutt - once this is available and we get some decent reprocessing we'll need to monitor these two quantities and include them in the performance studies.

This merge request affects 4 packages:

• Trigger/TrigSteer/TrigOutputHandling
• Trigger/TrigTools/IDScanZFinder
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @tamartin ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@nagano ,@rbielski ,@bernius as watchers

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

This merge request affects 4 packages:

• Trigger/TrigSteer/TrigOutputHandling
• Trigger/TrigTools/IDScanZFinder
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @tamartin ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@nagano ,@rbielski ,@bernius as watchers

CI Result FAILURE (hash b9b6b42c)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
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 34572]

CI Result FAILURE (hash 7e7cd65a)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
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 34573]

I do not think these are related issues. Should I restart CI or the fix is yet to arrive?

The same problem seems to appear in !44016 (merged) but is not known yet. RC suggested to restart CI once and open a jira ticket if it fails again, so will restart for now.

Cheers, Martin (L1)

Jenkins please retry a build

This merge request affects 4 packages:

• Trigger/TrigSteer/TrigOutputHandling
• Trigger/TrigTools/IDScanZFinder
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @tamartin ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@nagano ,@rbielski ,@bernius as watchers

CI Result SUCCESS (hash 7e7cd65a)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
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 34588]

CI succeeded now and changes look fine. Approving.

Cheers, Martin (L1)

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

mentioned in commit 203ff588

merged

added sweep:ignore label