Removed Physics folder, updated webdisplays, split track folder, formatted code.

Removed the physics folder from Data Quality (in histogram algorithm and webdisplays)

Went from Track/all histograms to Track/allTracks and Track/LeadTracks (changes in both histogram algorithm and webdisplays)

Updated the other webdisplays to be the same as the collisions_run

Tidied the format of code in tauMonitorAlgorithm.cxx

WARNING: big files (>100K) are found in the changeset

172K in file DataQuality/DataQualityConfigurations/config/Tau/collisions_minutes10.config

172K in file DataQuality/DataQualityConfigurations/config/Tau/heavyions_run.config

172K in file DataQuality/DataQualityConfigurations/config/Tau/collisions_run.config

This merge request affects 2 packages:

• DataQuality/DataQualityConfigurations
• Reconstruction/tauMonitoring

Affected files list will not be printed in this case

Adding @goetz as watcher

added DQ Reconstruction Tau master review-pending-level-1 labels

CI Result FAILURE (hash 51d9d40c)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 47570]

unit-tests failure:

      Start 570: DataQualityConfigurations_collisions_minutes10_syntax_ctest
 5/12 Test #568: DataQualityConfigurations_collisions_run_syntax_ctest ...........***Failed    6.88 sec
Testing han files for syntax and other problems.
To reproduce locally run "gmake test" in your build directory.
Using awk script /var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7

Hi @rferguso, could you please have a look?

Sergei L1

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

added 1 commit

• 9845bd56 - fixed configs

Compare with previous version

WARNING: big files (>100K) are found in the changeset

172K in file DataQuality/DataQualityConfigurations/config/Tau/collisions_minutes10.config

172K in file DataQuality/DataQualityConfigurations/config/Tau/heavyions_run.config

172K in file DataQuality/DataQualityConfigurations/config/Tau/collisions_run.config

This merge request affects 2 packages:

• DataQuality/DataQualityConfigurations
• Reconstruction/tauMonitoring

Affected files list will not be printed in this case

Adding @goetz as watcher

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

CI Result SUCCESS (hash 9845bd56)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 47841]

There are several instances of commented code that should probably be removed. Otherwise the changes look fine.

Jason (L1)

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

added 1 commit

• f6c4bd0e - removed commented code

Compare with previous version

WARNING: big files (>100K) are found in the changeset

172K in file DataQuality/DataQualityConfigurations/config/Tau/collisions_minutes10.config

172K in file DataQuality/DataQualityConfigurations/config/Tau/heavyions_run.config

172K in file DataQuality/DataQualityConfigurations/config/Tau/collisions_run.config

This merge request affects 2 packages:

• DataQuality/DataQualityConfigurations
• Reconstruction/tauMonitoring

Affected files list will not be printed in this case

Adding @goetz as watcher

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

CI Result SUCCESS (hash f6c4bd0e)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 48208]

Hi @rferguso, please resolve all open threads

Sergei L1

resolved all threads

Changes look good, previous comments have been addressed - approving.

Jakub (L1 shifter)

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

merged

mentioned in commit e59a7425

added sweep:ignore label