Skip to content

CA-tracking configuration per package: TrkParticleCreator

Thomas Strebler requested to merge tstreble/athena:TrkPartCreator_CA into 22.0

Continuation of updates of tracking CA configs, moving together the configs of the same C++ tools. This MR covers the TrkParticleCreator configs.

In the process, it was noticed that the configuration of the MuonParticleCreatorTool in MuonTrackBuildingConfig.py was pulling the config TrackParticleCreatorToolCfg from TrackRecoConfig.py, including in particular

kwargs.setdefault("BadClusterID", 3) # Select the mode to identify suspicous pixel cluster
kwargs.setdefault("KeepParameters", True)
kwargs.setdefault("KeepFirstParameters", False)

This differs from the RecExCommon config in MuonRecTools.py, which doesn't include those. The settings in the current CA config leads to many MuonSpectrometerTrackParticles variables to be set to dummy values, which is not the case with the RecExCommon config. This MR therefore restores a CA config for the MuonParticleCreatorTool compatible with the RecExCommon one, resulting in expected changes for the CA outputs of the type

Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixDiag.0.0 1.0 -> 16.051332473754883 => diff= [-44.13535569%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixDiag.0.1 1.0 -> 22.161455154418945 => diff= [-45.68248198%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixDiag.0.2 1.0 -> 2.7046964987675892e-06 => diff= [49.99972953%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixDiag.0.3 1.0 -> 2.766915940810577e-07 => diff= [49.99997233%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixDiag.0.4 1.0 -> 5.273389147228413e-13 => diff= [50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.0 0.0 -> -0.27343809604644775 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.1 0.0 -> -0.6627649664878845 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.2 0.0 -> 0.5445648431777954 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.3 0.0 -> -0.15948092937469482 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.4 0.0 -> 0.9128058552742004 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.5 0.0 -> 0.6049940586090088 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.6 0.0 -> -0.08929840475320816 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.7 0.0 -> 0.7153239250183105 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.8 0.0 -> 0.5946723818778992 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.definingParametersCovMatrixOffDiag.0.9 0.0 -> 0.875958263874054 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.phi.0 0.0 -> -1.3129613399505615 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.qOverP.0 0.0 -> -3.330223307784763e-06 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.theta.0 0.0 -> 2.6932671070098877 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.vx.0 0.0 -> 829.517333984375 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.vy.0 0.0 -> -3158.018310546875 => diff= [-50.00000000%]
Py:diff-root         INFO 001.MuonSpectrometerTrackParticlesAux.vz.0 0.0 -> -6785.0 => diff= [-50.00000000%]

(Unfortunately the values obtained after this fix do not match yet the ones obtained with the RecExCommon config.)

FYI @emoyse

Edited by Thomas Strebler

Merge request reports