Fix METRemappingAlg for PHYS->PHYSLITE production

The PHYS->PHYSLITE production workflow is currently generating an error in the METRemappingAlg due to the WriteHandles being checked for validity before they are recorded to. Simply reordering the isValid check to be after the record fixes this issue.

See JIRA: ATLASG-2653

Hi @akong ,

thanks a lot for preparing this! I think this can go into MR review now, it looks fine to me - I assume you tested that it works?

Cheers,

James

resolved all threads

marked this merge request as ready

Jenkins please retry a build

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss

This merge request affects 1 file:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss/src/METRemappingAlg.cxx

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

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

CI Result SUCCESS (hash 27c5fdca)

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

Looks good from my perspective.

Cheers, Anne (L1)

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

merged

mentioned in commit ddbc9986

changed the description