Creating JETM42 derivation (DADs)

This MR creates a new JETM42 derivation format based on the PHYS format that additionally includes clusters and towers as well as jets built from them. This derivation is intended for use in Upgrade studies.

@sch @htorre @mswiatlo @kleney @tamartin @tmhong

changed title from CReating JETM42 derivation (DADs) to Creating JETM42 derivation (DADs)

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss

Affected files list will not be printed in this case

Adding @jcatmore ,@ispokhar ,@emmat ,@jmellent ,@jveatch ,@nelsonc as watchers

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

CI Result SUCCESS (hash 245988e6)

	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 1
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9479] (remote access info)

Tagging @emmat for approval of a new derivation.

Switching tag while under discussion.

--Qiyu, L1

removed review-pending-level-1 label

added review-user-action-required label

resolved all threads

removed review-user-action-required label

added review-approved label

removed review-approved label

added review-pending-level-1 label

Just minor comments, giving it back to the user --L1

added review-user-action-required label

resolved all threads

added 1 commit

• bf665268 - removing comments, fixing copyright

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss

Affected files list will not be printed in this case

Adding @jveatch ,@jcatmore ,@emmat ,@nelsonc ,@jmellent ,@ispokhar as watchers

removed review-user-action-required label

CI Result SUCCESS (hash bf665268)

	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 1
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9599] (remote access info)

@drankin, please find some additional comments from the review. In particular, the compilation warnings caused by unused imports need to be fixed.

L1 shifter

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

added 1 commit

• 26092bc2 - removing unused imports

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss

Affected files list will not be printed in this case

Adding @jmellent ,@ispokhar ,@emmat ,@nelsonc ,@jveatch ,@jcatmore as watchers

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

added 1 commit

• 43cabb3c - removing unused imports

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss

Affected files list will not be printed in this case

Adding @emmat ,@jcatmore ,@jveatch ,@jmellent ,@nelsonc ,@ispokhar as watchers

@jkremer Thanks a lot for the comments! I have addressed them all with the recent commits.

CI Result SUCCESS (hash 26092bc2)

	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 [CI-MERGE-REQUEST-EL9 9679] (remote access info)

CI Result SUCCESS (hash 43cabb3c)

	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 [CI-MERGE-REQUEST-EL9 9680] (remote access info)

added Upgrade label

resolved all threads

Looks good, approving - L2

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

merged

mentioned in commit 6e7d6bb2