Refactoring ITk seed maker

I have refactored the ITk seed maker to provide clear measurement of the CPU contributions. In this way, the newEvent function is used to configure the environment and retrieve the space points while the production of the map storing the connections between space points happens when the find3sp function is called, just before seed production. This allows 1:1 comparison of CPU with the ACTS version of the same code.

No changes in performance are expected (both on CPU and physics). I have tested it running RunWorkflowTests_Run4.py.

FYI: @tstreble @cvarni @lfaldaul

added ITk Tracking master labels

assigned to @ncalace

build area is cleaned as the previous build was for the MR labeled as full-build

This merge request affects 1 package:

• InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk

This merge request affects 2 files:

• InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk/SiSpacePointsSeedTool_xk/ITkSiSpacePointsSeedMaker.h
• InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk/src/ITkSiSpacePointsSeedMaker.cxx

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

added InnerDetector main review-pending-level-1 labels

WARNING: big files (>100K) are found in the changeset

128K in file InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk/src/ITkSiSpacePointsSeedMaker.cxx

removed master label

resolved all threads

CI Result SUCCESS (hash 6e32f161)

	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-CC7 74502]

Just some small comments, otherwise looks good from an L1 point of view - L1

removed review-pending-level-1 label

added review-user-action-required label

resolved all threads

resolved all threads

removed review-user-action-required label

added review-pending-level-1 label

added 1 commit

• e3b27849 - updating

Compare with previous version

resolved all threads

This merge request affects 1 package:

• InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk

This merge request affects 2 files:

• InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk/SiSpacePointsSeedTool_xk/ITkSiSpacePointsSeedMaker.h
• InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk/src/ITkSiSpacePointsSeedMaker.cxx

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

WARNING: big files (>100K) are found in the changeset

128K in file InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk/src/ITkSiSpacePointsSeedMaker.cxx

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.

CI Result SUCCESS (hash e3b27849)

	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-CC7 74584]

Okay, thanks. The changes look okay, have been vetted a bit, and there are no issues with the CI. Approving from L1.

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

mentioned in commit 9b6811fc

merged

added sweep:ignore label

mentioned in merge request !64557 (merged)