Make InDetTimeCollection recording optional for ID raw data providers

Make InDetTimeCollection recording optional for all ID raw data providers and disable for trigger, since we weren't using the information in downstream algorithms. Additional minor cleanups (removal of default methods)

This merge request affects 3 packages:

• InnerDetector/InDetEventCnv/PixelRawDataByteStreamCnv
• InnerDetector/InDetEventCnv/SCT_RawDataByteStreamCnv
• InnerDetector/InDetEventCnv/TRT_RawDataByteStreamCnv

Affected files list will not be printed in this case

Adding @cgrefe ,@ibragimo ,@calfayan ,@lshan ,@goetz ,@stsuno ,@battagl ,@sroe as watchers

added 24.0 InnerDetector review-pending-level-1 labels

CI Result FAILURE (hash 5230e8a4)

	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 5437] (remote access info)

CI error looks related but the content of the MR looks like it shouldn't change anything in the output. L1

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

added 1 commit

• 4e47da2d - Add missing record() call, simplify logic

Compare with previous version

Thanks Lucy, yep, my fault, should be fixed with the latest commit

This merge request affects 3 packages:

• InnerDetector/InDetEventCnv/PixelRawDataByteStreamCnv
• InnerDetector/InDetEventCnv/SCT_RawDataByteStreamCnv
• InnerDetector/InDetEventCnv/TRT_RawDataByteStreamCnv

Affected files list will not be printed in this case

Adding @ibragimo ,@battagl ,@cgrefe ,@stsuno ,@calfayan ,@lshan ,@sroe ,@goetz as watchers

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

Hi @smh, does it address a particular problem or is it just a cleanup? cheers Jiri

CI Result SUCCESS (hash 4e47da2d)

	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 5450] (remote access info)

MR given back to the developer due to unresolved thread.

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-user-action-required label

resolved all threads

removed review-user-action-required label

added review-pending-level-1 label

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

Looks good to me. Approved.

Cheers L1

added review-approved-point1 label