ITSMFT: unify cellular automaton tracking for ITS and MFT by mpuccio · Pull Request #15781 · AliceO2Group/AliceO2

mpuccio · 2026-09-09T09:52:01Z

Introduce shared tracking, propagation, material handling and runtime ROF tables with compatibility wrappers for legacy ITS callers.

Add ITS and MFT CA workflows with checked configuration, reusable workflow sessions and workflow-owned publication. Include tracking and workflow tests.

Introduce shared tracking, propagation, material handling and runtime ROF tables with compatibility wrappers for legacy ITS callers. Add ITS and MFT CA workflows with checked configuration, reusable workflow sessions and workflow-owned publication. Include tracking and workflow tests.

alibuild · 2026-09-09T12:32:22Z

Error while checking build/O2/fullCI_slc9 for 41e1eba at 2026-09-09 14:32:

## 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
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Common/DCAFitter/GPU/cuda/GPUInterface.cu:55:15: error: use '= default' to define a trivial destructor [modernize-use-equals-default]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

ktf · 2026-09-13T15:45:29Z

@mpuccio the issue on ARM is most likely due floating point associative math. Can you change the check to be a - b < epsilon?

alibuild · 2026-09-15T05:59:15Z

Error while checking build/O2/fullCI_slc9 for e86650b at 2026-09-15 07:59:

## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Common/DCAFitter/GPU/cuda/GPUInterface.cu:55:15: error: use '= default' to define a trivial destructor [modernize-use-equals-default]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/common/tracking/test/testGenericTrack.cxx:835:66: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:111:32: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:114:31: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:140:43: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:148:25: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:152:32: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:259:16: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:355:52: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:359:50: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:365:86: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:368:32: error: statement should be inside braces [readability-braces-around-statements]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

Estimate the first q/pT from all attached hits with a weighted circle fit, using bounded storage and retaining the existing seed for field-off fits. Initialize both surface conventions with diagonal variances (1, 1, 1, 1, clamp(abs(q/pT), 1, 10)); inflate the previous diagonal by 25 and clear correlations for subsequent passes. Add signed-curvature, short-lever-arm and invalid-input coverage. Validated with run3-local3: 15 normalized-refit and 9 failure-contract cases pass, and all 346 assessment objects reproduce the preserved combined-circle MFT result exactly. Baseline fixture and staged library remain unchanged.

alibuild · 2026-09-22T09:05:08Z

Error while checking build/O2/fullCI_slc9 for f1e2189 at 2026-09-22 11:05:

## sw/BUILD/O2-latest/log
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/TPC/base/src/TPCBaseLinkDef.h:75:56
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/TPC/base/src/TPCBaseLinkDef.h:83:135
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/TPC/base/src/TPCBaseLinkDef.h:84:135
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/ECal/base/src/ECalBaseLinkDef.h:19:48
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/RICH/base/src/RICHBaseLinkDef.h:18:48
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/MID/base/src/MI3BaseLinkDef.h:18:47


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/ITS/workflow-ca/src/CATrackerSpec.cxx:193:77: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/ITS/workflow-ca/src/CATrackerSpec.cxx:200:16: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/MFT/workflow/src/CATrackerSpec.cxx:172:18: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/MFT/workflow/src/CATrackerSpec.cxx:206:77: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/MFT/workflow/src/CATrackerSpec.cxx:211:16: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:115:31: error: statement should be inside braces [readability-braces-around-statements]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

alibuild · 2026-09-22T09:43:32Z

Error while checking build/O2/fullCI_slc9 for e5c01a0 at 2026-09-23 13:09:

## sw/BUILD/O2-latest/log
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/TPC/base/src/TPCBaseLinkDef.h:75:56
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/TPC/base/src/TPCBaseLinkDef.h:83:135
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/TPC/base/src/TPCBaseLinkDef.h:84:135
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/ECal/base/src/ECalBaseLinkDef.h:19:48
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/MID/base/src/MI3BaseLinkDef.h:18:47
Error: missing ; at end of rule at /sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/RICH/base/src/RICHBaseLinkDef.h:18:48


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/GPU/GPUTracking/TRDTracking/GPUTRDTracker.cxx:700:42: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/GPU/GPUTracking/TRDTracking/GPUTRDTracker.cxx:707:42: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/ITS/workflow-ca/src/CATrackerSpec.cxx:193:77: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/ITS/workflow-ca/src/CATrackerSpec.cxx:200:16: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/MFT/workflow/src/CATrackerSpec.cxx:172:18: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/MFT/workflow/src/CATrackerSpec.cxx:206:77: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/MFT/workflow/src/CATrackerSpec.cxx:211:16: error: statement should be inside braces [readability-braces-around-statements]
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/TRKFT3/FT3/base/src/GeometryTGeo.cxx:115:31: error: statement should be inside braces [readability-braces-around-statements]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

alibuild · 2026-09-23T15:07:18Z

Error while checking build/O2/fullCI_slc9 for a847fab at 2026-09-24 09:09:

No log files found

Full log here.

mpuccio · 2026-09-24T12:39:27Z

Hi @ktf, if @f3sch does not object, I would merge this, fullCI ran yesterday and signaled some codecheck issues that I addressed in the last commit

f3sch · 2026-09-24T14:01:57Z

thanks @mpuccio, indeed it would be good to merge now since it does only touch little from a production point of view, it should be safe to merge and continue development on the dev branch.

mpuccio requested review from f3sch, fprino, mconcas, mcoquet642 and shahor02 as code owners September 9, 2026 09:52

mpuccio marked this pull request as draft September 9, 2026 09:52

alibuild mentioned this pull request Sep 9, 2026

Please consider the following formatting changes to #15781 mpuccio/AliceO2#59

Closed

mpuccio force-pushed the itsmft-tracking branch from 1d1d486 to 5a3a9a3 Compare September 9, 2026 10:01

mpuccio marked this pull request as ready for review September 9, 2026 10:09

mpuccio force-pushed the itsmft-tracking branch from 5a3a9a3 to 41e1eba Compare September 9, 2026 10:29

Fix tests for ARM CI

1afbfdc

alibuild mentioned this pull request Sep 14, 2026

Please consider the following formatting changes to #15781 mpuccio/AliceO2#60

Closed

Implement Maurice's comments

e86650b

mpuccio force-pushed the itsmft-tracking branch from 868a027 to e86650b Compare September 14, 2026 21:12

mpuccio added 3 commits September 15, 2026 15:34

Fix forward tracklet-compatibility selection

cab7ed4

Remove unused MFTFwdTrackHelpers

7d166bf

Remove operation result/failure metadata structs

42b4d4e

mpuccio marked this pull request as draft September 15, 2026 17:16

mpuccio added 2 commits September 15, 2026 22:13

Cleanup ITSSharedClusterCompatibility and output errors

a96d9fd

Rationalise the track publication

122c0d8

alibuild mentioned this pull request Sep 15, 2026

Please consider the following formatting changes to #15781 mpuccio/AliceO2#61

Closed

mpuccio added 2 commits September 16, 2026 09:04

Start migration of propagation bits into Propagator

da414ee

Rationalising IOUtils

c179073

mpuccio force-pushed the itsmft-tracking branch from dbd445d to c179073 Compare September 16, 2026 07:04

Clenaup configuration and track parameter handling

7a9af8a

mpuccio force-pushed the itsmft-tracking branch from cc26a6d to 7a9af8a Compare September 16, 2026 09:23

alibuild mentioned this pull request Sep 16, 2026

Please consider the following formatting changes to #15781 mpuccio/AliceO2#62

Closed

Unclutter detector layout description

73e450d

mpuccio force-pushed the itsmft-tracking branch from cb73653 to 73e450d Compare September 16, 2026 13:18

mpuccio added 5 commits September 16, 2026 19:07

Cell -> Triplet rename + cleanups

fd1df7a

Cleanup helpers for the Tracker

1b6f096

Cleanup indextable configuration

824fcf0

Using commond angle normalisation

f4a50d7

Unify application of systematics

69dde8b

alibuild mentioned this pull request Sep 16, 2026

Please consider the following formatting changes to #15781 mpuccio/AliceO2#63

Closed

mpuccio added 11 commits September 16, 2026 21:59

Unclutter the configuration

371f12c

Remove TrackingConfigParam duplication

427b86e

Further IOUtils cleanup

8b036bb

Consolidate the Propagator methods

4b93107

Use legacy timing classes

3c728a3

Remove timing helpers

1295950

Remove stale function

72b2899

Stabilise propagation in forward direction

c0d6334

Use fma to improve even further numerical stability

3820a39

Speedup circle fit

f1e2189

mpuccio force-pushed the itsmft-tracking branch from 6377631 to f1e2189 Compare September 22, 2026 08:15

mpuccio marked this pull request as ready for review September 22, 2026 08:31

Protect from invalid measurement covariances

e5c01a0

Fix checkcode errors

a847fab

Navigation Menu

ITSMFT: unify cellular automaton tracking for ITS and MFT - #15781

ITSMFT: unify cellular automaton tracking for ITS and MFT#15781
mpuccio wants to merge 30 commits into
AliceO2Group:devfrom
mpuccio:itsmft-tracking

mpuccio commented Sep 9, 2026

Uh oh!

alibuild commented Sep 9, 2026

Uh oh!

ktf commented Sep 13, 2026

Uh oh!

alibuild commented Sep 15, 2026

Uh oh!

alibuild commented Sep 22, 2026

Uh oh!

alibuild commented Sep 22, 2026 •

edited

Loading

Uh oh!

alibuild commented Sep 23, 2026 •

edited

Loading

Uh oh!

mpuccio commented Sep 24, 2026

Uh oh!

f3sch commented Sep 24, 2026 •

edited

Loading

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

4 participants

Conversation

mpuccio commented Sep 9, 2026

Uh oh!

alibuild commented Sep 9, 2026

Uh oh!

ktf commented Sep 13, 2026

Uh oh!

alibuild commented Sep 15, 2026

Uh oh!

alibuild commented Sep 22, 2026

Uh oh!

alibuild commented Sep 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alibuild commented Sep 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mpuccio commented Sep 24, 2026

Uh oh!

f3sch commented Sep 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

4 participants

alibuild commented Sep 22, 2026 •

edited

Loading

alibuild commented Sep 23, 2026 •

edited

Loading

f3sch commented Sep 24, 2026 •

edited

Loading