Skip to content

GPU TPC: Slightly augment dynamic buffer for TPC Decoding#15324

Open
cima22 wants to merge 1 commit intoAliceO2Group:devfrom
cima22:tpc_decoding_buffer_estimation
Open

GPU TPC: Slightly augment dynamic buffer for TPC Decoding#15324
cima22 wants to merge 1 commit intoAliceO2Group:devfrom
cima22:tpc_decoding_buffer_estimation

Conversation

@cima22
Copy link
Copy Markdown
Contributor

@cima22 cima22 commented Apr 24, 2026

Change first margin factor to 1.6 and add both factors to GPUMemorySizeScalers struct. Add 700 clusters per Sector Row for safety.

@cima22 cima22 requested a review from davidrohr as a code owner April 24, 2026 19:22
@cima22 cima22 changed the title GPU TPC: Slighly augment dynamic buffer for TPC Decoding GPU TPC: Slightly augment dynamic buffer for TPC Decoding Apr 24, 2026
@cima22 cima22 force-pushed the tpc_decoding_buffer_estimation branch from a0515a4 to 910ea93 Compare April 24, 2026 19:27
@alibuild
Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 910ea93 at 2026-04-24 23:50:

## sw/BUILD/O2-full-system-test-latest/log
command /sw/slc9_x86-64/O2/slc9_x86-64-slc9_x86-64-local17/prodtests/full-system-test/dpl-workflow.sh had nonzero exit code 128
[ERROR] Workflow crashed - PID 13418 (internal-dpl-aod-writer) did not exit correctly however it's not clear why. Exit code forced to 128.
[ERROR]  - Device internal-dpl-aod-writer: pid 13418 (exit 128)
[ERROR] SEVERE: Device internal-dpl-aod-writer (13418) returned with 128


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
grep: error-log.txt: binary file matches
++ GRERR=1
++ [[ 1 == 0 ]]
++ mkdir -p /sw/INSTALLROOT/ec8bd7d5e315d3b97ce1e7a596a7d2cedc12d57a/slc9_x86-64/o2checkcode/1.0-local118/etc/modulefiles
++ alibuild-generate-module
+ HAS_BIN=
+ HAS_LIB=
+ HAS_CMAKE=
+ HAS_ROOT=
+ '[' 0 '!=' 0 ']'
+ cat
+ printf 'if ![ is-loaded '\''BASE/1.0'\'' ] {\n module load BASE/1.0\n}'
+ echo 'BUILD_REQUIRES=CMake alibuild-recipe-tools defaults-release'
BUILD_REQUIRES=CMake alibuild-recipe-tools defaults-release
+ FULL_BUILD_REQUIRES='O2-customization Alice-GRID-Utils FairCMakeModules CMake UUID defaults-release bz2 googlebenchmark ninja Python-modules-list alibuild-recipe-tools date safe_int ninja-fortran json-c double-conversion MPFR pytorch_cpuinfo'
+ echo 'FULL_BUILD_REQUIRES=O2-customization Alice-GRID-Utils FairCMakeModules CMake UUID defaults-release bz2 googlebenchmark ninja Python-modules-list alibuild-recipe-tools date safe_int ninja-fortran json-c double-conversion MPFR pytorch_cpuinfo'
FULL_BUILD_REQUIRES=O2-customization Alice-GRID-Utils FairCMakeModules CMake UUID defaults-release bz2 googlebenchmark ninja Python-modules-list alibuild-recipe-tools date safe_int ninja-fortran json-c double-conversion MPFR pytorch_cpuinfo
++ env
++ cut -f1 -d=
++ grep -v '^DEFAULT_'
++ grep REVISION
++ sed -e s/_REVISION//
++ grep -v PKGREVISION
++ grep -v ALIBUILD_RECIPE_TOOLS
+ for x in $(env | cut -f1 -d= | grep -v "^DEFAULT_" | grep -v PKGREVISION | grep -v ALIBUILD_RECIPE_TOOLS | grep REVISION | sed -e 's/_REVISION//')
++ eval 'echo $LIBFFI_REVISION'
+++ echo 10
+ REVISION_VALUE=10
++ eval 'echo $LIBFFI_VERSION'
+++ echo v3.2.1-alice1
+ VERSION_VALUE=v3.2.1-alice1
++ eval 'echo $LIBFFI_ROOT'
+++ echo /sw/slc9_x86-64/libffi/v3.2.1-alice1-10
+ ROOT_PATH_VALUE=/sw/slc9_x86-64/libffi/v3.2.1-alice1-10
+ '[' -z 10 ']'
+ echo 'O2-customization Alice-GRID-Utils FairCMakeModules CMake UUID defaults-release bz2 googlebenchmark ninja Python-modules-list alibuild-recipe-tools date safe_int ninja-fortran json-c double-conversion MPFR pytorch_cpuinfo'
[0 more errors; see full log]

Full log here.

@cima22 cima22 force-pushed the tpc_decoding_buffer_estimation branch from 910ea93 to e5c13c2 Compare April 25, 2026 07:45
@shahor02
Copy link
Copy Markdown
Collaborator

Hi, can this be merged?

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.

3 participants