AFP webdisplay [rel 21]

AFP webdisplay

This is the first version of AFP webdisplay config file.
There are two algorithms Histogram_Not_Empty and BinContentComp.

---

@caforio @kristin

closed

changed the description

changed target branch from master to 21.0

reopened

This merge request affects 1 package:

• DataQuality/DataQualityConfigurations

added 21.0 DQ review-pending-level-1 labels

CI Result SUCCESS (hash 264f1e6a)

	Athena	AthDataQuality	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 161
AthDataQuality: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 6
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 44800]

The warnings are unrelated to the MR. Code looks fine to L1. Escalating to L2 for a second look as it is a new config file, Pienpen (L1)

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

This is a config file, there is no functionality to review. As for style, things are consistent and are easily readable. However, the copyright statement which must be in all files is missing. Passing back to the user to add the following lines

/*
  Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
*/

Cheers, Michele, L2 reviewer

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

added 1 commit

• f80c671a - Added copyright statement

Compare with previous version

This merge request affects 1 package:

• DataQuality/DataQualityConfigurations

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

CI Result SUCCESS (hash f80c671a)

	Athena	AthDataQuality	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthDataQuality: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 44815]

Already reviewed at L2, requested changes now implemented. Tests still pass. L1 approving.

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

merged

mentioned in commit 6ec2ea6a

added sweep:done label

picked the changes into the branch cherry-pick-6ec2ea6a5f0-master with commit 63320df0

mentioned in commit 63320df0

mentioned in merge request !36595 (merged)

Sweep summary
successful:

• master

(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/1286/console for details)

mentioned in commit 89675737

mentioned in merge request !36772 (merged)

mentioned in commit d71b36ab