Set useDynamicAlignFolders to true

Set this flag to True by default in master

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

116K in file InnerDetector/InDetExample/InDetRecExample/python/InDetJobProperties.py

This merge request affects 1 file:

• InnerDetector/InDetExample/InDetRecExample/python/InDetJobProperties.py

Adding @goetz ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

Hi @ponyisi, is this expected to affect tracking? This is a deprecated flag, right?

Hi @ponyisi, sorry but what is the purpose if this MR? I know the flag is supposed to be deprecated but usually turning this on by default crashes MC...

Hi @goblirsc , @npetters ,

I'm very happy to update the MR as necessary - but we have been talking about flipping the default of this flag (for data, at least) for ages, and forgetting to set it has caused us to trash multiple reprocessing campaigns in 21.0. We should not have to specify a preExec override for default reconstruction. (It also means that the q431 reconstruction test does not test the proper reconstruction code path.)

(for what it's worth the reason we didn't do this before in 21.0 is to observe the nominal frozen Tier-0 policy...)

Best, Peter

CI Result SUCCESS (hash f9501bf6)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22728]

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

resolved all threads

I resolved the thread as people seemed to be OK with the change.

Tadej (L2)

Adding ATLASRECTS-5400 which has a related discussion. @ponyisi could you update also useDynamicAlign in DetectorDescription/GeoModel/AtlasGeoModel/python/InDetGMJobProperties.py which should be the primary place to set it? cheers Jiri

added review-user-action-required label and removed review-approved label

@jmasik , all, should we just get rid of the InDetFlags version of this flag then?

added 1 commit

• 8a6e212e - Eliminate deprecated InDetFlags version of useDynamicAlignFolders

Compare with previous version

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

116K in file InnerDetector/InDetExample/InDetRecExample/python/InDetJobProperties.py

This merge request affects 6 files:

• Control/AthenaMonitoring/share/DataQualitySteering_jobOptions.py
• DetectorDescription/GeoModel/AtlasGeoModel/python/InDetGMJobProperties.py
• InnerDetector/InDetConfig/python/InDetConfigFlags.py
• InnerDetector/InDetExample/InDetRecExample/python/InDetJobProperties.py
• Reconstruction/RecExample/RecExOnline/share/RecExOnline_globalconfig_HI_2018.py
• Reconstruction/RecExample/RecExOnline/share/RecExOnline_globalconfig_pp_2018.py

Adding @goetz ,@sroe ,@nkoehler ,@ssnyder ,@rbianchi as watchers

added Core DQ Geometry Reconstruction review-pending-level-1 labels and removed review-user-action-required label

CI Result FAILURE (hash 8a6e212e)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22806]

added 1 commit

• dba71457 - Lock flag to false in MC

Compare with previous version

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

116K in file InnerDetector/InDetExample/InDetRecExample/python/InDetJobProperties.py