Avoid range error in SCT_ClusteringTool.

The SCT strip range computed from the number of the first strip and the group size may exceed the maximum strip number. To avoid range errors the strip range is capped.

Mitigates ATLASRECTS-6892.

This merge request affects 2 packages:

• InnerDetector/InDetConditions/SCT_ConditionsData
• InnerDetector/InDetRecTools/SiClusterizationTool

Affected files list will not be printed in this case

Adding @goetz ,@amorley ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash dd8170a6)

	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 1
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 1
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 48832]

The warnings when compiling Hijing_i are unrelated.

added 1 commit

• eb0e17a8 - Avoid range error in SCT_ClusteringTool.

Compare with previous version

resolved all threads

above changes: removed compile flag modifications (debugging leftover )

There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

This merge request affects 1 package:

• InnerDetector/InDetRecTools/SiClusterizationTool

This merge request affects 1 file:

• InnerDetector/InDetRecTools/SiClusterizationTool/src/SCT_ClusteringTool.cxx

Adding @goetz ,@amorley ,@sroe as watchers

CI Result SUCCESS (hash eb0e17a8)

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

the code changes look fine, CI passed, approving.

Roman(L1)

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

merged

mentioned in commit 767b9845

added sweep:ignore label