Fixes seed eta calculation.

Previously, seed eta parameters recorded in the validation ntuple were strictly positive:

This was due to a loss of sign from squaring and improper consideration of the bounds of std::atan. After this fix:

Tagging @lgagnon @pagessin

fyi @ncalace @tstreble

This merge request affects 1 package:

• InnerDetector/InDetRecEvent/SiSPSeededTrackFinderData

This merge request affects 1 file:

• InnerDetector/InDetRecEvent/SiSPSeededTrackFinderData/src/ITkSiSpacePointsProSeed.cxx

Adding @goetz ,@gavrilen ,@sroe ,@jsandesa as watchers

added InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash 1d5b50a2)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53411]

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

Changes and CI look fine. Approving.

Kira (L1)

mentioned in commit 0ffd856d

merged

added sweep:ignore label