DF SUSY Weights

The function to add the SoW for individual SUSY final states is no longer called on import of SUSYMetaData.py but has to be called explicitly with the Athena sequence as parameter. This function is also added to the SUSY-related part in addStandardTruthContents() of MCCommon, as it anyway relies on the decorations set by the already present DecorateSUSYProcess() before. All affected Derivation formats have been updated accordingly.

This also resolves a small bug in the DAOD_PHYS format, where addSUSYWeights() is run before DecorateSUSYProcess() and hence can't run properly.

Tagging also @crizzi and @zmarshal.

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkExamples
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMCTruth
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkSUSY

Adding @zmarshal ,@miholzbo ,@dshope ,@crizzi ,@alopezso as watchers

added 21.2 Derivation review-pending-level-1 labels

CI Result SUCCESS (hash 33e6b417)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 17
For experts only: Jenkins output [CI-MERGE-REQUEST-GCC62-CC7 119]

Two CI jobs are triggered by 21.2 MR events: for AthDerivation with gcc62 compiler and with gcc8 for AnalysisBase and AthAnalysis projects.

• Developers and shifters need to wait for both results, sometimes for additional hour or two.

CI Result SUCCESS (hash 33e6b417)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 6
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22081]

Thanks! Nice example of clean-up!

Looks good, warning are unrelated, approving --L1

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

@egramsta, @guirriec, @boeriu: This is only affecting derivations.

mentioned in commit a95b002b

merged

added sweep:ignore label

mentioned in merge request !37520 (merged)