Skip to content

[IOTOF] Update in the TOF digitization#15493

Draft
GiorgioAlbertoLucia wants to merge 16 commits into
AliceO2Group:devfrom
GiorgioAlbertoLucia:iotofdigi
Draft

[IOTOF] Update in the TOF digitization#15493
GiorgioAlbertoLucia wants to merge 16 commits into
AliceO2Group:devfrom
GiorgioAlbertoLucia:iotofdigi

Conversation

@GiorgioAlbertoLucia
Copy link
Copy Markdown
Contributor

@GiorgioAlbertoLucia GiorgioAlbertoLucia commented Jun 5, 2026

Main updates in the digitization

  • new values for the segmentation
  • the writer and reader class were fully implemented
  • the digitization is now done by first filling the chips, then streaming the digits in the chip to the output
  • the digitization reconstruction workflow was implemented and tested

The output digits was tested with a dedicated simulation

  • pions were injected with a gun to a target pixel
  • the map of the hits was visualized (as points) in the local coordinates of the chip
  • the map of the recorded digits was visualized both in the row/column and in the x/z local coordinates of the chip
  • the maps of the recorded digits and of the hits were superimposed to check that the two were consistent
Screenshot 2026-06-05 at 18 03 01 Map of the recorded digits in the row/column grid Screenshot 2026-06-05 at 18 03 17 Map of the recorded digits in the local coordinates of the chip Screenshot 2026-06-05 at 18 03 34 Superposition of the recorded digits and of the hits in the local coordinates of the chip Screenshot 2026-06-05 at 18 03 48 Zoom in

njacazio and others added 16 commits June 5, 2026 17:55
@njacazio @GiorgioAlbertoLucia
TOF digitiser
81c6889
@maciacco @GiorgioAlbertoLucia
fix digitizer workflow skeleton (use correct namespaces and tree colu…
dc9d987 
…mns)
@maciacco @GiorgioAlbertoLucia
fix issue in chip configuration (complex types cannot be used in conf…
2148157 
…igurables) + fix dictionary creation
@maciacco @GiorgioAlbertoLucia
remove debug print
557b717
@maciacco @GiorgioAlbertoLucia
add first information to digitizer spec and test reader
34e6113
@maciacco @GiorgioAlbertoLucia
add first element in workflow enabling digit processing
e85b558
@GiorgioAlbertoLucia
developing digitizer, added chips
8c47460
@maciacco @GiorgioAlbertoLucia
remove default constructor + fix compilation of digit and vector<digit>
71b45ac
@GiorgioAlbertoLucia
commit 1: all forseen branches available in the TF3 tree. Still preli…
c2fb078 
…minary approach to fill the Digits in the Digitizer class
@GiorgioAlbertoLucia
I forgor (one of the branches)
1a866c4
@maciacco @GiorgioAlbertoLucia
add tracking matrix cache
fb06b06
@GiorgioAlbertoLucia
commit 2: updated digitisation at the chip level (still need to figur…
d800da6 
…e out the proper way to clean up the extra digits)
@GiorgioAlbertoLucia
clang-format does not know how to handle CMakeLists apparently
f28379b
@maciacco @GiorgioAlbertoLucia
add first skeleton of reco workflow for iotof
51c194e
@maciacco @GiorgioAlbertoLucia
update segmentation parameters for updated geometry specs
4cd9b20
@maciacco @GiorgioAlbertoLucia
fix input name in digit reader + fix the branch types
0410c47
@alibuild alibuild mentioned this pull request Jun 5, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sawenzel sawenzel Awaiting requested review from sawenzel sawenzel will be requested when the pull request is marked ready for review sawenzel is a code owner

@shahor02 shahor02 Awaiting requested review from shahor02 shahor02 will be requested when the pull request is marked ready for review shahor02 is a code owner

@mconcas mconcas Awaiting requested review from mconcas mconcas will be requested when the pull request is marked ready for review mconcas is a code owner

@njacazio njacazio Awaiting requested review from njacazio njacazio will be requested when the pull request is marked ready for review njacazio is a code owner

@fcolamar fcolamar Awaiting requested review from fcolamar fcolamar will be requested when the pull request is marked ready for review fcolamar is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@GiorgioAlbertoLucia @njacazio @maciacco