Add a dedicated chi2 method.

This merge request affects 1 package:

• Tracking/TrkFitter/TrkGaussianSumFilterUtils

This merge request affects 1 file:

• Tracking/TrkFitter/TrkGaussianSumFilterUtils/src/GsfMeasurementUpdator.cxx

added Egamma Tracking main review-pending-level-1 labels

CI Result SUCCESS (hash 05847106)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 10202] (remote access info)

added 1 commit

• 33d4d6ad - Use chi2 method.

Compare with previous version

This merge request affects 1 package:

• Tracking/TrkFitter/TrkGaussianSumFilterUtils

This merge request affects 1 file:

• Tracking/TrkFitter/TrkGaussianSumFilterUtils/src/GsfMeasurementUpdator.cxx

CI Result SUCCESS (hash 33d4d6ad)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 10209] (remote access info)

added 1 commit

• f4575f25 - Move chi2 into Amg and use outside GsfMeasurementUpdator.

Compare with previous version

This merge request affects 4 packages:

• Event/EventPrimitives
• Tracking/TrkFitter/TrkGaussianSumFilterUtils
• Tracking/TrkTools/TrkMeasurementUpdator
• Tracking/TrkVertexFitter/TrkVertexBilloirTools

Affected files list will not be printed in this case

Adding @rbianchi ,@gemmeren as watchers

added EDM analysis-review-required labels

CI Result SUCCESS (hash f4575f25)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
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-EL9 10217] (remote access info)

removed review-pending-level-1 label

added 1 commit

• 7a58d00b - Add chi2 to CovarianceHelpers tests.

Compare with previous version

This merge request affects 4 packages:

• Event/EventPrimitives
• Tracking/TrkFitter/TrkGaussianSumFilterUtils
• Tracking/TrkTools/TrkMeasurementUpdator
• Tracking/TrkVertexFitter/TrkVertexBilloirTools

Affected files list will not be printed in this case

Adding @rbianchi ,@gemmeren as watchers

added review-pending-level-1 label

CI Result FAILURE (hash 7a58d00b)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
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-EL9 10220] (remote access info)

removed review-pending-level-1 label

added review-user-action-required label

CI failure is due to a changed test output however this is intentional.

ERROR: ../share/test_CovarianceHelpers.ref and test_CovarianceHelpers.log differ
...
The following tests FAILED:
	187 - EventPrimitives_test_CovarianceHelpers_ctest (Failed)

@christos This is a bit closer to what you had in mind?

yes

Perhaps with some comment from the 3.2.1 of Frühwirth . More or less saying this is the the chi2 statistic of the Least Squares estimator etc

This is a unit test so the dev can update the references ....