21.9: Porting changes to fast clusterisation (as done in master) to profit of it by default.

Since I have updated the version for master (!36138 (merged)), I think would be useful to back-port it for 21.9, so that we can run the fast pixel clustering while preserving the output.

Changes in the RunUpgradeTests.py as the pixel clusters are sorted differently in ESDs and AODs. I have run IDPVM to compare the cluster quantities after running RunUpgradeTests.py using reference and patched versions and the results are 1:1 identical: https://test-dcube.web.cern.ch/test-dcube/dcube_test_clustering/

Tagging potentially interested people: @nstyles, @tstreble, @iene

added 21.9 label

This merge request affects 2 packages:

• InnerDetector/InDetExample/InDetRecExample
• InnerDetector/InDetRecTools/SiClusterizationTool

Adding @goetz ,@oda ,@amorley ,@sroe as watchers

added InnerDetector review-pending-level-1 labels

changed title from Porting changes to fast clusterisation (as done in master) to profit of it by default. to 21.9: Porting changes to fast clusterisation (as done in master) to profit of it by default.

CI Result SUCCESS (hash d1b5705d)

	Athena
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 149
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 45045]

Looks good, just minor comment, passing to L2 for a second opinion --L1

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

Do not see any additional problems here... @ncalace , could you answer the L1 comment ?

 Pavol [as L2 MR shifter]

added review-user-action-required label

added 15 commits

• d1b5705d...fa223e86 - 13 commits from branch atlas:21.9
• eab8d66d - changing at into []
• fb93dba1 - Merge branch '21.9' of https://gitlab.cern.ch:8443/atlas/athena into 21.9-updating-clustering

Compare with previous version

This merge request affects 2 packages:

• InnerDetector/InDetExample/InDetRecExample
• InnerDetector/InDetRecTools/SiClusterizationTool

Adding @goetz ,@oda ,@amorley ,@sroe as watchers

added review-pending-level-1 label and removed review-pending-level-2 review-user-action-required labels

resolved all threads

CI Result SUCCESS (hash fb93dba1)

	Athena
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 45077]

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

merged

mentioned in commit 56e6d8eb

added sweep:ignore label