Add CA configuration for DESDM_MCP & DRAW_ZMUMU

• Add CA configuration for the DESDM_MCP & DRAW_ZMUMU file formats (Tagging: @pscholer, @rosati, @cgrefe, @gfacini)
• Transform the MuonAnalysisThinningTool to an reentrant algorithm
• AsgSelectionToolWrapper declares the data dependencies
• Add various CA snippets to DerivationFrameworkTools

Tagging: @jcatmore, @tadej Closes ATLASRECTS-7423

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMuons
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTools

Affected files list will not be printed in this case

Adding @apsallid ,@jcatmore ,@maklein ,@mvanadia ,@gabarone ,@calpigia as watchers

added Derivation master labels

marked this merge request as draft

changed title from WIP: Add CA configuration for DESDM_MCP to Draft: Add CA configuration for DESDM_MCP

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)AF3.*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result FAILURE (hash de1ebf39)

	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-CC7 64585]

added 1 commit

• 6245b71a - Define DESDM_MCP output

Compare with previous version

added 64 commits

• 6245b71a...4b61cb16 - 60 commits from branch atlas:master
• 73fdf48d - Added TriggerSkimmingTool to common config
• f50d7a91 - Transform the AnalysisMuonThinningTool to an Algorithm
• 57288d04 - Define DESDM_MCP output
• 26264a2f - First draft

Compare with previous version

Jenkins please retry a build

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMuons
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTools
• PhysicsAnalysis/PrimaryDPDMaker
• Reconstruction/RecJobTransforms

Affected files list will not be printed in this case

Adding @apsallid ,@jcatmore ,@maklein ,@mvanadia ,@gabarone ,@calpigia ,@goetz as watchers

added Analysis Reconstruction labels

CI Result FAILURE (hash 26264a2f)

	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 2
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 64672]

added 70 commits

• 26264a2f...394c5a1c - 64 commits from branch atlas:master
• 9fc94ee2 - Added TriggerSkimmingTool to common config
• 011f86e8 - Transform the AnalysisMuonThinningTool to an Algorithm
• 1e7a540a - Define DESDM_MCP output
• d42fa4b8 - First draft
• 4b8f5f22 - Use data handles
• 34db4eb1 - Add DRAW Zmumu configuration

Compare with previous version

added 109 commits

• 34db4eb1...d70d4c25 - 100 commits from branch atlas:master
• 73440d7a - Added TriggerSkimmingTool to common config
• 4d37b03f - Transform the AnalysisMuonThinningTool to an Algorithm
• d70b4290 - Define DESDM_MCP output
• 4bed8e0f - First draft
• a182f159 - Use data handles
• e3aebe66 - Add DRAW Zmumu configuration
• 2493f8c2 - Blub Blub
• 0fe9887e - More fixes
• e4eb22cf - Hupps renamed the file

Compare with previous version

Jenkins please retry a build

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMuons
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTools
• PhysicsAnalysis/PrimaryDPDMaker
• Reconstruction/RecJobTransforms

Affected files list will not be printed in this case

Adding @apsallid ,@jcatmore ,@maklein ,@mvanadia ,@gabarone ,@calpigia ,@goetz as watchers

CI Result FAILURE (hash e4eb22cf)

	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-CC7 64834]

added 15 commits

• e4eb22cf...c92f8746 - 6 commits from branch atlas:master
• 2d3961ac - Added TriggerSkimmingTool to common config
• c730fa53 - Transform the AnalysisMuonThinningTool to an Algorithm
• ec58b7a6 - Define DESDM_MCP output
• 7f90e606 - First draft
• be6268aa - Use data handles
• a912f542 - Add DRAW Zmumu configuration
• 68d5dc55 - Blub Blub
• 862bcb13 - More fixes
• 47c55cc4 - Hupps renamed the file

Compare with previous version

added 27 commits

• 47c55cc4...faf4e23b - 26 commits from branch atlas:master
• df540f7c - Add DESDM_MCP & DRAW zmumu to CA

Compare with previous version

marked this merge request as ready

changed title from Draft: Add CA configuration for DESDM_MCP to Add CA configuration for DESDM_MCP & DRAW_ZMUMU

added 1 commit

• a67abaa1 - Remove tool

Compare with previous version

changed the description

added NewConfig label

resolved all threads

Build area was cleaned as per request posted in the DB. The full software build will be performed

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMuons
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTools
• PhysicsAnalysis/PrimaryDPDMaker
• Reconstruction/RecJobTransforms

Affected files list will not be printed in this case

Adding @apsallid ,@jcatmore ,@maklein ,@mvanadia ,@gabarone ,@calpigia ,@goetz as watchers

added review-pending-level-1 label

Build area was cleaned as per request posted in the DB. The full software build will be performed

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMuons
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTools
• PhysicsAnalysis/PrimaryDPDMaker
• Reconstruction/RecJobTransforms

Affected files list will not be printed in this case

Adding @apsallid ,@jcatmore ,@maklein ,@mvanadia ,@gabarone ,@calpigia ,@goetz as watchers

CI Result FAILURE (hash df540f7c)

	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-CC7 64900]

CI Result FAILURE (hash a67abaa1)

	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-CC7 64902]

added 1 commit

• 1b951d79 - Wondering why this doesn't work

Compare with previous version

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMuons
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTools
• PhysicsAnalysis/PrimaryDPDMaker
• Reconstruction/RecJobTransforms

Affected files list will not be printed in this case

Adding @apsallid ,@jcatmore ,@maklein ,@mvanadia ,@gabarone ,@calpigia ,@goetz as watchers

CI Result SUCCESS (hash 1b951d79)

	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-CC7 64922]

The changes look fine and there are no CI issues. Approving.

Jason (L1)

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

mentioned in commit 6d19c8da

merged

added sweep:ignore label