Skip to content

Conversation

@ktf
Copy link
Member

@ktf ktf commented Jul 1, 2025

No description provided.

@ktf ktf requested a review from a team as a code owner July 1, 2025 09:22
@github-actions
Copy link
Contributor

github-actions bot commented Jul 1, 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

@ktf ktf changed the title DPL: make sure devices with occasional inputs (Timers, Sporadic) get sorted last, if possible DPL: make sure devices with Sporadic inputs get sorted last, if possible Jul 1, 2025
@ktf
Copy link
Member Author

ktf commented Jul 1, 2025

@shahor02 updated. The timers have to be sorted first in general, of course.

@shahor02
Copy link
Collaborator

shahor02 commented Jul 1, 2025

W/o QC it runs, but with QC I get

44493:qc-task-TRD-RawData]: [17:17:58][ERROR] Exception caught while in Init: Unable to find requested service N2o29framework15CallbackServiceE with hash 151729959 using salt 0 for service kind 4. Exiting with 1.

Reproducible also with FST

@ktf
Copy link
Member Author

ktf commented Jul 1, 2025

Mmm... It looks unrelated to me, though. I will investigate in a bit.

@shahor02
Copy link
Collaborator

shahor02 commented Jul 1, 2025

You are right, reverting your PR I still get it. Will update my QC.

@shahor02
Copy link
Collaborator

shahor02 commented Jul 1, 2025

OK, after rebuilding the QC it runs with your PR, I've processed ~100 TFs w/o problems. The new sortint seems to be fine, I see 2 QC tasks forwarding to aod-producer, but they have no sporadic inputs:

grep to_aod-producer-workflow dump.txt
                "from_ctp-entropy-decoder_to_aod-producer-workflow",
                "from_MIDTracker_to_aod-producer-workflow",
                "from_trd-pulseheight_to_aod-producer-workflow",
                "from_hmp-matcher_to_aod-producer-workflow",
                "from_muon-track-matcher_to_aod-producer-workflow",
                "from_pvertex-track-matching_to_aod-producer-workflow",
                "from_secondary-vertexing_to_aod-producer-workflow",
                "from_tpc-time-series_to_aod-producer-workflow",
                "from_qc-task-HMP-Matches_to_aod-producer-workflow",
                "from_qc-task-GLO-MUONTracks_to_aod-producer-workflow",

Just in case, attaching the topology dump.
dump.txt.gz

@ktf
Copy link
Member Author

ktf commented Jul 1, 2025

ok, let's merge. If I understand correctly next daily will have GPU support and we can do a test.

@ktf ktf merged commit 4cfc7e4 into AliceO2Group:dev Jul 1, 2025
17 checks passed
@ktf ktf deleted the pr14459 branch July 1, 2025 19:51
SchrodingersGhostCat pushed a commit to SchrodingersGhostCat/AliceO2 that referenced this pull request Jul 4, 2025
mhemmer-cern pushed a commit to mhemmer-cern/AliceO2 that referenced this pull request Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants