Skip to content

Conversation

@sawenzel
Copy link
Contributor

@sawenzel sawenzel commented Oct 2, 2025

While studying file-access graphs in O2DPG MC, I noticed that trdtracklets.root get written twice: Once from the digitizer workflow and once from the TRD-DIGI-QC task. This can corrupt the processing and is probably a bug. QC should only read digits/trdtracklets.

The reason appears to be the use of o2-trd-trap-sim as the reader command for digits and tracklets.

For now, the problem can probably be avoided by using o2-trd-trap-sim --disable-root-output true.

However, this might still create new trdtracklets in memory which should not be needed as they have been previously created during digitization.

So it would be good to just use a true o2-trd-sim-artefacts-reader workflow or similar in the future.

While studying file-access graphs in O2DPG MC, I noticed
that trdtracklets.root get written twice: One from the digitizer workflow
and once from the TRD-DIGI-QC task. This can corrupt the processing and is probably a bug.
QC should only read digits/trdtracklets.

The reason appears to be the use of `o2-trd-trap-sim` as the reader command for
digits and tracklets.

For now, the problem can probably be avoided by using `o2-trd-trap-sim --disable-root-output true`.

However, this might still create new trdtracklets in memory which should not be needed
as they have been previously created during digitization.

So it would be good to just use a true `o2-trd-sim-artefacts-reader` workflow or similar in the future.
@sawenzel sawenzel requested a review from jackal1-66 as a code owner October 2, 2025 09:33
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@sawenzel sawenzel merged commit 40625bf into AliceO2Group:master Oct 2, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant