added context to interfaces for VKalVrtFitter, and passes via EMVertexBuilder

We're adjusting the interfaces and starting to pass down the EventContext from algs to VKalVrtFitter. Using EMVertexBuilder as first alg.

@christos @amoussa

This merge request affects 4 packages:

• InnerDetector/InDetRecTools/InDetConversionFinderTools
• Reconstruction/egamma/egammaAlgs
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexFitterInterfaces

Adding @goetz ,@amorley ,@sroe as watchers

added Egamma InnerDetector Reconstruction Tracking master review-pending-level-1 labels

added 53 commits

• 9dbbdcb6...5a2294ba - 52 commits from branch atlas:master
• 039d3388 - removed TrackParticleBaseCollection method from InDetConversionFinderTools

Compare with previous version

This merge request affects 4 packages:

• InnerDetector/InDetRecTools/InDetConversionFinderTools
• Reconstruction/egamma/egammaAlgs
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexFitterInterfaces

Adding @goetz ,@amorley ,@sroe as watchers

CI Result SUCCESS (hash 9dbbdcb6)

	Athena	AthSimulation	AnalysisBase	AthGeneration
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 7
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15409]

CI Result SUCCESS (hash 039d3388)

	Athena	AthSimulation	AnalysisBase	AthGeneration
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 7
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15419]

added 1 commit

• a3213ced - fix compiler warnings

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetRecTools/InDetConversionFinderTools
• Reconstruction/egamma/egammaAlgs
• Tracking/TrkVertexFitter/TrkV0Fitter
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexBilloirTools
• Tracking/TrkVertexFitter/TrkVertexFitterInterfaces
• Tracking/TrkVertexFitter/TrkVertexFitters

Adding @goetz ,@bouhova ,@amorley ,@sroe as watchers

CI Result SUCCESS (hash a3213ced)

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

@schaffer . Sorry but several copyrights need to be fixed. Thanks BA as L1

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

added 1 commit

• fbbc2a24 - fixing copyright

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetRecTools/InDetConversionFinderTools
• Reconstruction/egamma/egammaAlgs
• Tracking/TrkVertexFitter/TrkV0Fitter
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexBilloirTools
• Tracking/TrkVertexFitter/TrkVertexFitterInterfaces
• Tracking/TrkVertexFitter/TrkVertexFitters

Adding @goetz ,@bouhova ,@amorley ,@sroe as watchers

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

CI Result SUCCESS (hash fbbc2a24)

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

The changes look fine to me and there are no CI issues. Approving.

Jason (L1)

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

merged

mentioned in commit 27e50c38

added sweep:ignore label