New configurations

Add new configurations TRTPhaseConfig.py, TRTPreProcessing.py, TrackingCommonConfig.py, all configurations are working!

This merge request affects 1 package:

• InnerDetector/InDetConfig

Adding @goetz ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash 0d2aa17d)

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

I only have a comment on commented code.

The MR description mentions that these configurations are working. Is it possible to had tests to prove it and make sure future development has a baseline to work with?

Cheers,

Michele (L1 reviewer)

Sorry for the Late Reply. Here are attached configuration pkl and log files for run procedure. Cheers, Zaza test_TRTPhaseConfig.pkl

test_TRTPrepocessing.pkl

TRTPhaseConfig_Run_Log.txt

TRTPreProcessing_Run_Log.txt

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

added 1 commit

• d6bb072f - replace TRT_On to new flag, remove outdated properties and TrackingCommonConfig.py not used yet

Compare with previous version

This merge request affects 1 package:

• InnerDetector/InDetConfig

Adding @goetz ,@sroe as watchers

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

CI Result SUCCESS (hash d6bb072f)

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

resolved all threads

Thanks for considering my comments. All green, approving.

Cheers,

Michele

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

merged

mentioned in commit 0ad2945e

added sweep:ignore label