Do not create views at offline level but only RoIs

This will replace the ACTS Event View maker with a simple algorithm for creating an RoI for later use in (e.g.) the clustering

Current sequence:

added ACTS label

changed the description

/cc @adye @bkerridg

This merge request affects 6 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @tstreble ,@cvarni ,@sroe ,@lgagnon ,@jburzyns ,@jojungge ,@pagessin ,@bkerridg ,@tbold ,@toyamaza ,@keli ,@goetz as watchers

added InnerDetector Tracking main review-pending-level-1 labels

added 1 commit

• 73d64431 - remove unwanted changes

Compare with previous version

added full-unit-tests label

This merge request affects 6 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @keli ,@tbold ,@cvarni ,@lgagnon ,@sroe ,@goetz ,@jburzyns ,@tstreble ,@toyamaza ,@bkerridg ,@pagessin ,@jojungge as watchers

added 1 commit

• 4571aa19 - data prep unit tests

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsDataPreparation
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @bkerridg ,@pagessin ,@tbold ,@goetz ,@toyamaza ,@sroe ,@keli ,@jojungge ,@jburzyns ,@tstreble ,@lgagnon ,@cvarni as watchers

added 1 commit

• 87697a5c - Small fixes

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsDataPreparation
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @tbold ,@bkerridg ,@jojungge ,@tstreble ,@goetz ,@pagessin ,@cvarni ,@jburzyns ,@toyamaza ,@keli ,@sroe ,@lgagnon as watchers

CI Result FAILURE (hash c5d3684c)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6861] (remote access info)

CI Result FAILURE (hash 73d64431)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6862] (remote access info)

CI Result FAILURE (hash 4571aa19)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6863] (remote access info)

CI Result FAILURE (hash 87697a5c)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6865] (remote access info)

Escalating to L2 due to major restructuring of the Code.

~L1

removed review-pending-level-1 label

added review-pending-level-2 label

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

marked this merge request as draft

added 2 commits

• cf0bae4d - unit test
• 4f92ef60 - Merge remote-tracking branch 'origin/DoNotCreateViews' into DoNotCreateViews

Compare with previous version

marked this merge request as ready

added 1 commit

• 63a02b4f - propagate roi name

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsDataPreparation
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @tstreble ,@keli ,@bkerridg ,@tbold ,@sroe ,@jojungge ,@cvarni ,@goetz ,@pagessin ,@toyamaza ,@lgagnon ,@jburzyns as watchers

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

added Run3-MCReco-output-changed label

CI Result FAILURE (hash 63a02b4f)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6933] (remote access info)

CI Failures are unrelated

removed full-unit-tests label

removed Run3-MCReco-output-changed label

Jenkins please retry a build

This merge request affects 7 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsDataPreparation
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @tbold ,@sroe ,@bkerridg ,@lgagnon ,@goetz ,@pagessin ,@jburzyns ,@jojungge ,@toyamaza ,@keli ,@tstreble ,@cvarni as watchers

CI Result FAILURE (hash 63a02b4f)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6998] (remote access info)

added 1 commit

• 8f8e43ba - add namespace for comp factory

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsDataPreparation
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @cvarni ,@pagessin ,@keli ,@jojungge ,@tbold ,@jburzyns ,@tstreble ,@toyamaza ,@lgagnon ,@bkerridg ,@sroe ,@goetz as watchers

CI Result FAILURE (hash 8f8e43ba)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 7004] (remote access info)

added 1 commit

• 264e5357 - no roi for tool

Compare with previous version

This merge request affects 7 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• Tracking/Acts/ActsConfig
• Tracking/Acts/ActsDataPreparation
• Tracking/Acts/ActsRegionsOfInterest
• Tracking/Acts/ActsToolInterfaces
• Tracking/Acts/ActsView

Affected files list will not be printed in this case

Adding @keli ,@lgagnon ,@cvarni ,@toyamaza ,@tbold ,@sroe ,@jburzyns ,@goetz ,@bkerridg ,@pagessin ,@jojungge ,@tstreble as watchers

CI Result SUCCESS (hash 264e5357)

	Athena	AthSimulation
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
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 7012] (remote access info)

MR looks fine from L1 perspective, CI shows no problems - approving.

Cheers, L1 shifter

removed review-pending-level-1 label

added review-approved label

mentioned in commit 023d47a7

merged