Skip to content

Comments

GPU TPC: Fix minor bug in cluster removal protection and minor improvements, fixes the deterministic mode partially#15086

Open
davidrohr wants to merge 4 commits intoAliceO2Group:devfrom
davidrohr:PR
Open

GPU TPC: Fix minor bug in cluster removal protection and minor improvements, fixes the deterministic mode partially#15086
davidrohr wants to merge 4 commits intoAliceO2Group:devfrom
davidrohr:PR

Conversation

@davidrohr
Copy link
Collaborator

This fixes for sure the problem we introduced into the deterministic mode with the cluster removal protection.
However, on the NVIDIA Blackwell GPU I now see a new inconsistency in the sector track refit.
That code was not changed in a long time, so it could be an artifact of CUDA 13 / the new NVIDIA architecture.
Will debug and fix in a separate PR.

@github-actions
Copy link
Contributor

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

@alibuild
Copy link
Collaborator

Error while checking build/O2/fullCI_slc9 for f638a7f at 2026-02-20 06:41:

++ cp /sw/slc9_x86-64/O2/15086-slc9_x86-64-local1/compile_commands.json .
+++ python3 -c 'import json, os; print(os.path.commonpath([x["file"] for x in json.loads(open("compile_commands.json").read()) if "sw/BUILD" not in x["file"] and "G__" not in x["file"] and x["file"].endswith(".cxx")]))'
++ O2_SRC=/sw/SOURCES/O2/15086-slc9_x86-64/0
++ [[ -e /sw/SOURCES/O2/15086-slc9_x86-64/0/CMakeLists.txt ]]

Full log here.

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