Sweeping !59861 from master to 21.2. Change default setting in muon SF top CP tool

Change default setting in muon SF top CP tool

See merge request !59861 (merged)

added sweep:from master label

Adding original author @hawatson as watcher.

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools

This merge request affects 1 file:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools/Root/TopMuonCPTools.cxx

Adding @sahinsoy ,@omajersk ,@ravinab ,@thsteven ,@tdado as watchers

added 21.2 Analysis AnalysisTop analysis-review-required review-pending-level-1 labels

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

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools

This merge request affects 1 file:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools/Root/TopMuonCPTools.cxx

Adding @sahinsoy ,@omajersk ,@ravinab ,@thsteven ,@tdado as watchers

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

There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

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.

Two CI jobs are triggered by 21.2 MR events: for AthDerivation with gcc62 compiler and with gcc8 for AnalysisBase and AthAnalysis projects.

• Developers and shifters need to wait for both results, sometimes for additional hour or two.

CI Result SUCCESS (hash f9c8b286)

	AnalysisBase
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
AnalysisBase: number of compilation errors 0, warnings 1
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 64682]

CI Result SUCCESS (hash f9c8b286)

	AthDerivation
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
AthDerivation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-GCC62-CC7 2100]

Hi @hawatson I see the following failure in the CI

01/18/2023 17:36 :: ERROR    :: Py:diff-root      WARNING the following variables exist only in the new file !

01/18/2023 17:36 :: ERROR    :: Py:diff-root      WARNING  - [EventInfoAuxDyn.DFCommonJets_clusterMETy]

01/18/2023 17:36 :: ERROR    :: Py:diff-root      WARNING  - [EventInfoAuxDyn.DFCommonJets_clusterMETx]

01/18/2023 17:36 :: ERROR    :: Your tag breaks changes the AOD content! See patched_derivation_test/diff-root.DAOD.log file for more information.

Does the change of the config flag introduced with this MR explain it? If so, there is a ref file to update

-- L1

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

Jenkins please retry a build

resolved all threads

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools

This merge request affects 1 file:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools/Root/TopMuonCPTools.cxx

Adding @sahinsoy ,@omajersk ,@ravinab ,@thsteven ,@tdado as watchers

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

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools

This merge request affects 1 file:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools/Root/TopMuonCPTools.cxx

Adding @sahinsoy ,@omajersk ,@ravinab ,@thsteven ,@tdado as watchers

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.