Tie HGTD dict building to DetFlag

It was noticed today by @tstreble that unfortunately having the HGTD switched on by default in the dictionary generation breaks reconstruction from existing RDOs (e.g. those from the phys val campaign). Indeed, I see differences in the Identifiers written to output when running RunUpgradeTests.py and toggling this on/off, and so while jobs complete successfully and the results look normal when starting from HITS (because Identifiers are then consistent throughout), we can still see the effect that causes problems when starting from RDOs (where they are not, and so no tracks are found).

I've therefore tied the generation of the HGTD part of the dictionary to the HGTD_on DetFlag so that things will remain consistent for the moment, and the differences will only come in once we try to actually include the HGTD in the geometry fully.

FYI @dshope @cohm @asalzbur @ncalace

This merge request affects 1 file:

• InnerDetector/InDetExample/InDetSLHC_Example/python/SLHC_Setup_ITK.py

Adding @goetz ,@sroe as watchers

added 21.9 InnerDetector review-pending-level-1 labels

CI Result SUCCESS (hash f40262df)

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

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

merged

mentioned in commit 8d6f69bc

added sweep:ignore label

Demonstration of strip and pixel Identifiers before/after: