Use di-tau smart slimming instead of AllVariables

Hello,

As reported in !27694 (merged) , the introduction of a smart slimming list for DiTauJets and DiTauJetsLowPt collections has caused trouble for DAOD formats using AllVariables for these collections. The use of AllVariables is not allowed for large formats. The smart collections have to be used instead. As far as I can see, only DFHiggs needs that change. TAUP5 still uses AllVariables for DiTauJetsLowPt, but it's a small CP format which is not subject to the AllVariables restriction. I am running ART tests with the fix for HDBS1 and HIGG4D2, it seems fine.

Cheers, Bertrand

added 21.2 Derivation labels

added urgent label

I'm setting the "urgent" flag in case this fix would be required by derivation coordination before building the new cache.

mentioned in merge request !27694 (merged)

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkHiggs

Adding @oda as watcher

added review-pending-level-1 label

@oda , could you please verify you are happy with the changes?

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 41a77b6d)

	AthDerivation
externals
cmake
make
required tests
optional tests

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

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 41a77b6d)

	AnalysisBase	AnalysisTop	AthAnalysis
externals
cmake
make
required tests
optional tests

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

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

I just have a minor comment to the code. Magdalena (L1)

added 1 commit

• 98781fc5 - very minor

Compare with previous version

resolved all threads

removed review-user-action-required label

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkHiggs

Adding @oda as watcher

added review-pending-level-1 label

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 98781fc5)

	AnalysisBase	AnalysisTop	AthAnalysis
externals
cmake
make
required tests
optional tests

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

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 98781fc5)

	AthDerivation
externals
cmake
make
required tests
optional tests

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

assigned to @ntamir

unassigned @ntamir

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

looks fine to me.

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

merged

mentioned in commit e0af86ce

added sweep:ignore label