Fix: Save Pixel & Strip Clusters to AOD for Phase-II Geometry

This draft of a MR resolves an issue related to saving Pixel and Strip clusters to AOD files when using samples based on the ATLAS Phase-II geometry. The problem arises due to:

• Container name differences: Pixel/Strip cluster container names differ between Phase-I and Phase-II, but are currently hardcoded;
• Different preparation functions: The process for preparing Pixel/Strip clusters for AOD varies between the two geometries.

Changes & Fixes

• Introduced two new flags to specify Pixel/Strip container names, defaulting to the current naming convention;
• Added a switch to detect the geometry version and import the correct preparation functions accordingly.

These changes are minimal and have been tested successfully with both Run 3 and Run 4 samples.

requested review from @pibutti, @dguest, and @goblirsc

added 1 commit

• c4b9e2bc - Thanks for the suggestions. I agree: this looks better.

Compare with previous version

added 1 commit

• 6c5beb54 - Fixed indentation; changed pixel/strip container names

Compare with previous version

resolved all threads

marked this merge request as ready

This merge request affects 3 packages:

• PhysicsAnalysis/JetTagging/JetHitAssociation
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagConfig

Affected files list will not be printed in this case

Adding @bdong ,@ligang ,@lidiaz ,@dguest ,@robouque ,@svanstro as watchers

added Analysis BTagging Derivation JetEtmiss main review-pending-level-1 labels

CI integration tests for projects Athena are cancelled because of compilation error(s)

CI Result FAILURE (hash 6c5beb54)

	Athena	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 1, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

added 1 commit

• 62007078 - Standardized Pixel/Strip container names across all instances

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/JetTagging/JetHitAssociation
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagConfig

Affected files list will not be printed in this case

Adding @robouque ,@bdong ,@lidiaz ,@dguest ,@ligang ,@svanstro as watchers

CI Result SUCCESS (hash 62007078)

	Athena	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 1
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

added 1 commit

• d64639a9 - Removed unused import to eliminate compiler warning

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/JetTagging/JetHitAssociation
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagConfig

Affected files list will not be printed in this case

Adding @robouque ,@svanstro ,@dguest ,@lidiaz ,@bdong ,@ligang as watchers

CI Result SUCCESS (hash d64639a9)

	Athena	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
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

resolved all threads

added 1 commit

• eab9c93e - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/JetTagging/JetHitAssociation
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagConfig

Affected files list will not be printed in this case

Adding @bdong ,@ligang ,@dguest ,@robouque ,@lidiaz ,@svanstro as watchers

resolved all threads

CI Result SUCCESS (hash eab9c93e)

	Athena	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
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

added 1 commit

• 61a1d3d9 - Replaced Pixel/Strip container flags by automated configuration based on geometry version

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/JetTagging/JetHitAssociation
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

Affected files list will not be printed in this case

Adding @bdong ,@svanstro ,@lidiaz ,@dguest ,@robouque ,@ligang as watchers

resolved all threads

CI Result SUCCESS (hash 61a1d3d9)

	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 (remote access info)

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit bc360659