24.2 Updating AT example config and input file names

The current example config and input file names for AnalysisTop don't nicely describe what these files are used for.

This MR adapts the file names to reflect the example files used for Run2/Run3 and data/MC.

It adds corresponding comments and links to the R22 and R24 changelogs to the config files.

changed title from Draft: 24.2 updating at config and input file names to Draft: 24.2 Updating AT example config and input file names

marked this merge request as ready

jenkins please retry a build

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis

This merge request affects 8 files:

• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config-Run2.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config-Run3.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config-mc21.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data18-rel22.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data18-rel24.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data22-rel22.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data22-rel24.txt

Adding @skorn ,@cgarvey ,@ravinab ,@thsteven as watchers

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

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

CI Result SUCCESS (hash 81116705)

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

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

Looks good to me. Approved.

Cheers L1

@jnitschk After a quick discussion within the AT Reco team we decided to add a few more changes to this MR.

setting review-user-action-required

removed review-approved label

added review-user-action-required label

added 1 commit

• 773193d9 - Updating naming convention for AT CI files

Compare with previous version

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis

This merge request affects 18 files:

• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/scripts/CI_EMPFlowData22test.py
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/scripts/CI_EMPFlowDatatest.py
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/scripts/CI_EMPFlowMC21test.py
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/scripts/CI_EMPFlowMCtest.py
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/CI-validation-cuts-Run2.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/CI-validation-cuts-Run2.txt~
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/CI-validation-cuts-Run3
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/CI-validation-cuts-Run3~
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config-Run2.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config-Run3.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config-mc21.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/example-config.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data18-rel22.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data18-rel24.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data22-rel22.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/input-data22-rel24.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/validation-cuts-mc21.txt
• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis/share/validation-cuts.txt

Adding @skorn ,@ravinab ,@mhodgkin ,@cgarvey ,@thsteven as watchers

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

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

CI Result SUCCESS (hash 773193d9)

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

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

Still looks good to me. Approved.

Cheers L1

added analysis-review-approved label and removed analysis-review-required label

mentioned in commit cf8c531c

merged

mentioned in merge request !65666 (merged)

added sweep:ignore label