Skip to content

add check in TOF compressor before to save RDH open - #13389

Merged
noferini merged 1 commit into
AliceO2Group:devfrom
noferini:dev
Aug 17, 2024
Merged

add check in TOF compressor before to save RDH open#13389
noferini merged 1 commit into
AliceO2Group:devfrom
noferini:dev

Conversation

@noferini

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown
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-apass3
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0

@noferini noferini changed the title [WIP] add check in TOF compressor before to save RDH open add check in TOF compressor before to save RDH open Aug 16, 2024
@noferini

Copy link
Copy Markdown
Collaborator Author

This should fix an issue spotted in
https://its.cern.ch/jira/browse/O2-5231

in case of an issue in processTRM rewind didn't reset also start of buffer pointer.
In case of decoding error not-resetted starting pointer was used to replace current buffer point moving it outside the buffer

adding @preghenella

@noferini
noferini requested a review from davidrohr August 16, 2024 14:05
@noferini
noferini enabled auto-merge (squash) August 17, 2024 07:48
@noferini
noferini merged commit c64fb16 into AliceO2Group:dev Aug 17, 2024
maciacco pushed a commit to maciacco/AliceO2 that referenced this pull request Aug 22, 2024
Co-authored-by: noferini <9963644+noferini@users.noreply.github.com>
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