New DD configuration: introduced an overrider for the standard path to SQLite Geometry DB

Introduced a new command-line argument for job transforms --geometrySQLiteFullPath to override the standard path to the SQLite Geometry Database. If this argument is used then GeoModelSvc switches to the initialization of geometry from SQLite and takes the SQLite database path from the override instead of trying to find the file in the standard location

CC @sroe @nstyles @tadej @jojungge @boudreau

mentioned in merge request !68245 (merged)

This merge request affects 5 packages:

• Control/AthenaConfiguration
• DetectorDescription/GeoModel/AtlasGeoModel
• DetectorDescription/GeoModel/GeoModelSvc
• Simulation/SimuJobTransforms
• Tools/PyJobTransforms

Affected files list will not be printed in this case

Adding @apsallid ,@jchapman ,@jojungge ,@maszyman ,@ssnyder ,@gemmeren ,@rbianchi as watchers

added Core Geometry Simulation Tools analysis-review-required full-unit-tests main review-pending-level-1 labels

added full-integration-tests label

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

CI Result SUCCESS (hash e4d5cc1c)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
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
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4166]

Hi @tsulaia, Thanks for the update.

In the description, you wrote

a new command-line argument for job transforms --geometrySQLiteFullPath

I was wondering about VP1.

Would that command line argument be reflected by/through an Athena flag/option as well?

That would be needed for tools like VP1 that do not use the transforms but configure and run Athena on the fly.

Thanks, --Ric.

resolved all threads

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

MR looks good I am approving - AR

removed analysis-review-required label

added analysis-review-approved label

MR looks good to me, approving.

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-approved label

mentioned in commit 3f9e74fc

merged