remove DBM in the output tables of InDetRecStatistics

• remove DBM in the tables.

This merge request affects 1 package:

• InnerDetector/InDetValidation/InDetRecStatistics

Adding @goetz ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash b1c1c221)

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

Hi @hye, please either make a boolean flag or remove the lines completely. We want to avoid commented-out code.

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

I think we might want in certain cases just rename the eta-region DBM to Forward or something similar. when its concerning counting hits in the DBM, we can completely remove these statements.

Hi @npetters and @tadej , do you mean we just rename the eta-region DBM currently, not remove DBM?

Hi @hye, just when the DBM represent an eta-region, we should rename it to forward. when its concerning counting hits in the DBM, we can completely remove these statements.

added 1 commit

• eacff6e2 - rename DBM as forward in the eta region, remove DBM in the column of the hits table

Compare with previous version

This merge request affects 1 package:

• InnerDetector/InDetValidation/InDetRecStatistics

Adding @goetz ,@sroe as watchers

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

Hi @npetters and @tadej , I rename DBM as forward when it means eta region and remove DBM in the hits table. Please have a look.

CI Result SUCCESS (hash eacff6e2)

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

resolved all threads

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

mentioned in commit bad4e677

merged

added sweep:ignore label