ONNX to ringer chains

Include the ONNX ringer code. E/g hypo can use the old or new (ONNX) ringer selector now.

Trigger count changes it is expected since we updated the ringer tuning for low energy triggers.

Link ATLINFR-3994

changed the description

This merge request affects 3 packages:

• PhysicsAnalysis/RingerSelectorTools
• Trigger/TrigHypothesis/TrigEgammaHypo
• Trigger/TrigHypothesis/TrigMultiVarHypo

Affected files list will not be printed in this case

Adding @dbakshig ,@cjmeyer ,@fernando as watchers

added Analysis Egamma Trigger master review-pending-level-1 labels

@jenkins please retry a build

Hi @jodafons ,

I hope it passes but it would be better to use the onnxruntime service from TrigEgammaFastCaloHypoToolInc.h instead of RingerSelectorTools (which is not a Tool in reality) I did some researches on this to make it work with onnxruntime let me know if you want to discus.

Thanks, Debo.

changed title from ONNX to ringer chains to WIP ONNX to ringer chains

changed title from WIP ONNX to ringer chains to ONNX to ringer chains

added 1 commit

• fc4180b9 - update path

Compare with previous version

CI Result SUCCESS (hash 3b57b88b)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 29153]

This merge request affects 3 packages:

• PhysicsAnalysis/RingerSelectorTools
• Trigger/TrigHypothesis/TrigEgammaHypo
• Trigger/TrigHypothesis/TrigMultiVarHypo

Affected files list will not be printed in this case

Adding @dbakshig ,@cjmeyer ,@fernando as watchers

changed the description

CI Result FAILURE (hash fc4180b9)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 29159]

added 1 commit

• 86b81769 - update

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/RingerSelectorTools
• Trigger/TrigHypothesis/TrigEgammaHypo
• Trigger/TrigHypothesis/TrigMultiVarHypo

Affected files list will not be printed in this case

Adding @dbakshig ,@cjmeyer ,@fernando as watchers

CI Result FAILURE (hash 86b81769)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 29160]

Trigger counts have changed in Trigger_athena_data-test and Trigger_athena_MC-test. If this is OK, the references need to be updated -- L1

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

added changes-trigger-counts label

added 2 commits

• 7184c1be - switch to onnx new tunings
• 6a092fa4 - update counts

Compare with previous version