IDC: Adjust random IDC access in various packages 4

Adjusting usage of IDC for better random access.

WARNING: big files (>100K) are found in the changeset

116K in file MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools/src/MuonLayerHoughTool.cxx

This merge request affects 12 packages:

• InnerDetector/InDetEventCnv/InDetEventCnvTools
• InnerDetector/InDetEventCnv/InDetEventTPCnv
• InnerDetector/InDetMonitoring/SCT_Monitoring
• InnerDetector/InDetRecEvent/SiSPSeededTrackFinderData
• InnerDetector/InDetRecTools/InDetAssociationTools
• InnerDetector/InDetRecTools/TRT_SeededSpacePointFinderTool
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_DAF
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_xk
• InnerDetector/InDetTrigRecAlgs/InDetTrigPrepRawDataFormat
• InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
• MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools
• MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MuonPatternSegmentMaker

Adding @goetz ,@cgrefe ,@amorley ,@wleight ,@gavrilen ,@sroe ,@nagano ,@rosati ,@bernius ,@calfayan ,@oda ,@sutt ,@nkoehler as watchers

added DQ InnerDetector MuonSpectrometer Trigger master review-pending-level-1 labels

CI Result FAILURE (hash ba059d8f)

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

removed review-pending-level-1 label

added review-user-action-required label

added 1 commit

• 60e206f6 - Fix bugs - changes iterator names when not an iterator

Compare with previous version

WARNING: big files (>100K) are found in the changeset

116K in file MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools/src/MuonLayerHoughTool.cxx

This merge request affects 12 packages:

• InnerDetector/InDetEventCnv/InDetEventCnvTools
• InnerDetector/InDetEventCnv/InDetEventTPCnv
• InnerDetector/InDetMonitoring/SCT_Monitoring
• InnerDetector/InDetRecEvent/SiSPSeededTrackFinderData
• InnerDetector/InDetRecTools/InDetAssociationTools
• InnerDetector/InDetRecTools/TRT_SeededSpacePointFinderTool
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_DAF
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_xk
• InnerDetector/InDetTrigRecAlgs/InDetTrigPrepRawDataFormat
• InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
• MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools
• MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MuonPatternSegmentMaker

Adding @goetz ,@cgrefe ,@amorley ,@wleight ,@gavrilen ,@sroe ,@nagano ,@rosati ,@bernius ,@calfayan ,@oda ,@sutt ,@nkoehler as watchers

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

CI Result FAILURE (hash 60e206f6)

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

added 2 commits

• 00882526 - Adjust IDC random access in various packages
• fb0c6066 - Fix a double delete problem in FastOnlineIDC

Compare with previous version

WARNING: big files (>100K) are found in the changeset

116K in file MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools/src/MuonLayerHoughTool.cxx

This merge request affects 13 packages:

• Event/EventContainers
• InnerDetector/InDetEventCnv/InDetEventCnvTools
• InnerDetector/InDetEventCnv/InDetEventTPCnv
• InnerDetector/InDetMonitoring/SCT_Monitoring
• InnerDetector/InDetRecEvent/SiSPSeededTrackFinderData
• InnerDetector/InDetRecTools/InDetAssociationTools
• InnerDetector/InDetRecTools/TRT_SeededSpacePointFinderTool
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_DAF
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_xk
• InnerDetector/InDetTrigRecAlgs/InDetTrigPrepRawDataFormat
• InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
• MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools
• MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MuonPatternSegmentMaker

Adding @goetz ,@cgrefe ,@amorley ,@wleight ,@gavrilen ,@sroe ,@nagano ,@rosati ,@bernius ,@calfayan ,@oda ,@sutt ,@nkoehler as watchers

added EDM label

CI Result FAILURE (hash fb0c6066)

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

Jenkins please retry a build

WARNING: big files (>100K) are found in the changeset

116K in file MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools/src/MuonLayerHoughTool.cxx

This merge request affects 13 packages:

• Event/EventContainers
• InnerDetector/InDetEventCnv/InDetEventCnvTools
• InnerDetector/InDetEventCnv/InDetEventTPCnv
• InnerDetector/InDetMonitoring/SCT_Monitoring
• InnerDetector/InDetRecEvent/SiSPSeededTrackFinderData
• InnerDetector/InDetRecTools/InDetAssociationTools
• InnerDetector/InDetRecTools/TRT_SeededSpacePointFinderTool
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_DAF
• InnerDetector/InDetRecTools/TRT_TrackExtensionTool_xk
• InnerDetector/InDetTrigRecAlgs/InDetTrigPrepRawDataFormat
• InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
• MuonSpectrometer/MuonReconstruction/MuonPatternFinders/MuonPatternFinderTools/MuonHoughPatternTools
• MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MuonPatternSegmentMaker

Adding @goetz ,@cgrefe ,@amorley ,@wleight ,@gavrilen ,@sroe ,@nagano ,@rosati ,@bernius ,@calfayan ,@oda ,@sutt ,@nkoehler as watchers

CI Result FAILURE (hash fb0c6066)

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

resolved all threads

The changes look fine to me, but I am passing this to L2 for another look.

Jason (L1)

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

Fine by me.

Tadej (L2)

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

merged

mentioned in commit 62035637

added sweep:ignore label