TrigValTools: Add rootcomp unit test and cleanup

• add a unit test for rootcomp.py
• enable flake8 for all python files and make them compliant
• delete trigslimval.py, TRoot2Html and related files (ATR-20090)

This merge request affects 1 package:

• Trigger/TrigValidation/TrigValTools

Adding @okumura ,@jpanduro as watchers

added Trigger master review-pending-level-1 labels

Thanks, looks good!

CI Result FAILURE (hash a94b8683)

	Athena	AthSimulation	AthGeneration
externals
cmake
make
required tests
optional tests

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

Jenkins please retry a build

This merge request affects 1 package:

• Trigger/TrigValidation/TrigValTools

Adding @okumura ,@jpanduro as watchers

CI Result SUCCESS (hash a94b8683)

	Athena	AthSimulation	AthGeneration
externals
cmake
make
required tests
optional tests

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

The changes look fine to me. Passing to L2 for further review.

Jason (L1)

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

Looks good, approving.

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

merged

mentioned in commit 689b9371

added sweep:ignore label

mentioned in merge request !35522 (merged)