Add pixel clustering number network lwtnn implementation in r22

Following up on @kpachal's implementation of lwtnn for the position networks here, this implements a similar functionality for the number network.

Tagging @gfacini, @battagl, @npetters, @vcairo, @goblirsc, for info.

changed title from WIP: Add pixel clustering number network lwtnn implementation in master to WIP: Add pixel clustering number network lwtnn implementation in r22

added 1 commit

• f3d39ebb - Remove unnecessary printouts

Compare with previous version

changed the description

unmarked as a Work In Progress

This merge request affects 1 package:

• InnerDetector/InDetRecTools/SiClusterizationTool

Adding @goetz ,@oda ,@amorley ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

added 168 commits

• 6b6ead8c...df70e094 - 167 commits from branch atlas:master
• f818fa7a - Merge remote-tracking branch 'upstream/master' into master-srettie_NNdev

Compare with previous version

CI Result SUCCESS (hash f3d39ebb)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21273]

hi @srettie , back to user since there are unresolved threads.

cheers, Santiago (L1)

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

added 263 commits

• f3d39ebb...ea3e6642 - 261 commits from branch atlas:master
• ad8c4481 - Add proper error statements
• 1f74a1ce - Merge remote-tracking branch 'upstream/master' into master-srettie_NNdev

Compare with previous version

build area is cleaned as the previous build was for the MR labeled as full-build

This merge request affects 1 package:

• InnerDetector/InDetRecTools/SiClusterizationTool

Adding @goetz ,@oda ,@amorley ,@sroe as watchers

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

CI Result SUCCESS (hash 1f74a1ce)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21541]

Still one unresolved thread - sending back to user.

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

resolved all threads

resolved all threads

resolved all threads

added review-pending-level-1 label

removed review-user-action-required label

Hi @dgil,

I had forgotten to resolve the thread, sorry about that! I've resolved them now, so please feel free to proceed with the review.

Cheers.

Sébastien.

resolved all threads

No problem, thanks.

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

merged

mentioned in commit caf17486

added sweep:ignore label