RoadPredictor tool for Trigger Phase II ITk tracking, EFTRACK-142

added a new tool prototype, development for ITk Phase II trigger tracking as described in EFTRACK-142, no changes in test output expected.

This merge request affects 2 packages:

• Trigger/TrigTools/TrigInDetToolInterfaces
• Trigger/TrigTools/TrigInDetTrackFitter

Affected files list will not be printed in this case

Adding @sutt ,@keli ,@nagano ,@bkerridg as watchers

added Trigger TriggerID main review-pending-level-1 labels

CI Result SUCCESS (hash c64e9360)

	Athena
externals
cmake
make
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3022]

added EFTracking ITk labels

Some very minor code formatting issues mentioned in threads but otherwise looks good.

Cheers, Lucy (L1)

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

added review-user-action-required label and removed review-approved label

added 1 commit

• 96806096 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• b4bb1419 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

This merge request affects 2 packages:

• Trigger/TrigTools/TrigInDetToolInterfaces
• Trigger/TrigTools/TrigInDetTrackFitter

Affected files list will not be printed in this case

Adding @sutt ,@bkerridg ,@nagano ,@keli as watchers

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

resolved all threads

Build area was cleaned as per request posted in the DB. The full software build will be performed

This merge request affects 2 packages:

• Trigger/TrigTools/TrigInDetToolInterfaces
• Trigger/TrigTools/TrigInDetTrackFitter

Affected files list will not be printed in this case

Adding @sutt ,@bkerridg ,@keli ,@nagano as watchers

CI Result SUCCESS (hash 96806096)

	Athena
externals
cmake
make
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3052]

CI Result SUCCESS (hash b4bb1419)

	Athena
externals
cmake
make
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3053]

I've just shared a few extra comments worth to consider.

Best, Thomas - L2

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

added 1 commit

• 9b47922e - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• 6d475441 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

This merge request affects 2 packages:

• Trigger/TrigTools/TrigInDetToolInterfaces
• Trigger/TrigTools/TrigInDetTrackFitter

Affected files list will not be printed in this case

Adding @keli ,@sutt ,@bkerridg ,@nagano as watchers

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

This merge request affects 2 packages:

• Trigger/TrigTools/TrigInDetToolInterfaces
• Trigger/TrigTools/TrigInDetTrackFitter

Affected files list will not be printed in this case

Adding @keli ,@sutt ,@nagano ,@bkerridg as watchers

resolved all threads

CI integration tests for projects Athena are cancelled because of compilation error(s)

CI Result FAILURE (hash 9b47922e)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 1, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3104]

CI Result SUCCESS (hash 6d475441)

	Athena
externals
cmake
make
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3105]

The changes have been vetted already and there are no issues with the CI. Approving from L1.

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

mentioned in commit 6663b17d

merged