Add Btagging and additional variable to HIGG1D1

Corrects the calibration sequence for jets in Higg1D1, Adds b-tagging and a number of e/gamma decorations to egammna objects in the HIGG1D1 derivation.

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkCalo
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkHiggs
• Reconstruction/Jet/JetCalibTools
• Reconstruction/Jet/JetMomentTools

Affected files list will not be printed in this case

Adding @goetz ,@dshope ,@mdaneri ,@mleblanc as watchers

added Derivation JetEtmiss Reconstruction analysis-review-required master review-pending-level-1 labels

CI Result SUCCESS (hash 12719b4d)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 52745]

Looks OK from analysis review side, approving

Giovanni (AR)

removed analysis-review-required label

added analysis-review-approved label

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

Only two minor comments regarding print statements from L1 point of view. Passing on to L2 in the meantime for a cross check. It would also be good to have a sign off from the HIGG derivation contacts but I didn't find who's responsible.

Kira (L1)

added 1 commit

• 3686a73b - Remove debugging print statements and add a small comment

Compare with previous version

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkCalo
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkHiggs
• Reconstruction/Jet/JetCalibTools
• Reconstruction/Jet/JetMomentTools

Affected files list will not be printed in this case

Adding @goetz ,@dshope ,@mdaneri ,@mleblanc as watchers

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

resolved all threads

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

Putting L2 label back as before because L1 comments have been adressed.

Kira (L1)

Jenkins please retry a build

Job restarted after the intervention for a Java version update

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkCalo
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkHiggs
• Reconstruction/Jet/JetCalibTools
• Reconstruction/Jet/JetMomentTools

Affected files list will not be printed in this case

Adding @goetz ,@dshope ,@mdaneri ,@mleblanc as watchers

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

CI Result SUCCESS (hash 3686a73b)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 52838]

Once again passing to L2 shifter after CI re-run.

Jakub (L1 shifter)

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

Looks good, approving.

Sergio (L2)

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

merged

mentioned in commit dcfc5ea4

added sweep:ignore label