Combining the thinning tools for PHYS and PHYSLITE into a single config

ATLASG-2390

This MR combines the (currently separate) thinning configs for PHYS and PHYSLITE into a common config script. This will help to keep them in synch, and simplifies the requirement that the config supports both AOD->PHYSLITE and PHYS->PHYSLITE workflows. It also reduces duplicated config code.

added Derivation NewConfig master labels

added 1 commit

• 42237982 - Adjusting PHYS to include the new config script

Compare with previous version

added 1 commit

• 6e023a0b - Fixing incorrect stream name for PHYS->PHYSLITE (should be D2AOD not DAOD)

Compare with previous version

marked this merge request as ready

changed the description

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys

This merge request affects 3 files:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys/python/PHYS.py
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys/python/PHYSLITE.py
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys/python/PhysCommonThinningConfig.py

Adding @zmarshal ,@jcatmore ,@calpigia ,@dshope as watchers

added review-pending-level-1 label

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 SUCCESS (hash 6e023a0b)

	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 63178]

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

Jason (L1)

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

added review-approved-point1 label

merged

mentioned in commit 2e13dfb6

added sweep:ignore label