[IOTOF]: Updates in the TOF digitization for ALICE3#15492
Closed
GiorgioAlbertoLucia wants to merge 107 commits into
Closed
[IOTOF]: Updates in the TOF digitization for ALICE3#15492GiorgioAlbertoLucia wants to merge 107 commits into
GiorgioAlbertoLucia wants to merge 107 commits into
Conversation
…igurables) + fix dictionary creation
developing digitizer, added chips
…minary approach to fill the Digits in the Digitizer class
…e out the proper way to clean up the extra digits)
Please consider the following formatting changes to AliceO2Group#15492
Collaborator
…to two halves (AliceO2Group#15395) * adjustments for service z and r positions, split OT barrel into two halves * reverting back OT stave tilting angle * Please consider the following formatting changes --------- Co-authored-by: Igor Altsybeev <Igor.Altsybeev@cern.ch> Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…d ROF staggering) (AliceO2Group#15351) * ALICE3: move tracking to GlobalReconstruction * ALICE3: add cluster and GPU tracking paths * ALICE3: derive ROF timing for TRK inputs * ALICE3: write track ROFs and IR frames * ALICE3: assign cluster MC labels in TimeFrame loading * ALICE3: share TimeFrame code between CPU and GPU * ALICE3: validate configuration before running * ALICE3: clang-format * ALICE3: fix initialisation with MC vertices Co-authored-by: Copilot <copilot@github.com> * ALICE3: add new performance macro * ALICE3: Fix check cluster macro * ALICE3: add protections in reading clusters --------- Co-authored-by: Copilot <copilot@github.com>
Due to the ensureContinuousROF some stray ROFs with clusters may be eliminated, which was not accounted in the cluster vector.
…#15397) * Fix sensitive volumes name for FT3 * Removed unused variables
There is no point in using the slow boundary checking / buffer resizing cursor when we reserve, since it's reasonable to expect that the memory will be there.
…eader (AliceO2Group#15482) * Add TPC/CMVTrigger spec to TFReader * Account for the DD headers convention (one per multipart)
…Group#15481) * Use qTot as marker for HIP clusters instead of qMax * Adjust handling of saturated qTot * Adjust qTot limit in cluster statistics
Please consider the following formatting changes to AliceO2Group#15492
Collaborator
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Main updates:
the digitization is done by storing the information from the hits to the chips first. Then the digits are constructed
the digitizer reader and writer were updated to store all the relevant information to the tree (previously commented)
the full digitization procedure was finished and tested
a simulation was run locally to test the writing of digits: