DerivationFrameworkCore: Add an algorithm to explicitly lock a decoration.
In derivation, we have cases where the same decoration is written by several different algorithms (for example, TruthDressingTool). In such cases, we cannot lock the decoration at the end of the algorithm, as is usually done, since this would cause subsequent algorithms to fail. However, we also don't want to leave the decoration unlocked, as the fix for ATLASRECTS-8008 would cause those decorations to be lost in a deep copy. Instead, in those cases, we can use this algorithm to explicitly lock the decoration after all algorithms that modify it have completed.
These configurations were previously working due to a bug in WritDecorHandle, where it would fail to lock a decoration if isAvailable() had been called. Fixing that bug necessitates this change.
Note, though, that such configurations are likely not compatible with MT.
In support of !73627 (closed) / ATLASRECTS-8008.
Merge request reports
Activity
This merge request affects 1 package:
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkCore
This merge request affects 4 files:
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkCore/CMakeLists.txt
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkCore/src/LockDecoration.cxx
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkCore/src/LockDecoration.h
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkCore/src/components/DerivationFrameworkCore_entries.cxx
added 24.0 Derivation review-pending-level-1 labels
CI Result SUCCESS (hash 5eb9cf01)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)removed review-pending-level-1 label
added review-approved label
mentioned in commit 6f284c43
added review-approved-tier0 label
mentioned in merge request !74399 (merged)