Use EventContext in SCT_SiliconConditionsTool

As thread local variable retrieval is slow when calling Gaudi::Hive::currentContext() try to use EventContext in SCT_SiliconConditionsTool in an effort to speed-up digitisation.

/cc @oda

This merge request affects 6 packages:

• InnerDetector/InDetConditions/InDetConditionsSummaryService
• InnerDetector/InDetConditions/SCT_ConditionsAlgorithms
• InnerDetector/InDetConditions/SCT_ConditionsTools
• InnerDetector/InDetConditions/SiLorentzAngleTool
• InnerDetector/InDetConditions/SiPropertiesTool
• InnerDetector/InDetDigitization/SCT_Digitization

Affected files list will not be printed in this case

Adding @goetz ,@jchapman ,@sroe as watchers

added Digitization InnerDetector master review-pending-level-1 labels

CI Result FAILURE (hash bc2eecae)

	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 1, 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 27821]

added 1 commit

• 48c52402 - Compilation fiz

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetConditions/InDetConditionsSummaryService
• InnerDetector/InDetConditions/SCT_ConditionsAlgorithms
• InnerDetector/InDetConditions/SCT_ConditionsTools
• InnerDetector/InDetConditions/SiLorentzAngleTool
• InnerDetector/InDetConditions/SiPropertiesTool
• InnerDetector/InDetDigitization/SCT_Digitization
• InnerDetector/InDetExample/InDetDetDescrExample

Affected files list will not be printed in this case

Adding @goetz ,@jchapman ,@sroe as watchers

CI Result SUCCESS (hash 48c52402)

	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 27856]

Looks good but can you correct "perpandicular" with "perpendicular" every time in this code? Thanks (L1).

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

This is out of scope of the MR and I'd prefer to avoid another iteration.

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

Sure, maybe in the future.

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

Hi @csebasti,

Can you replace "perpandicular" with "perpendicular" once this MR is merged?

Thanks in advance, Susumu

merged

mentioned in commit 376bf419

added sweep:ignore label