Skip to content

ITS Online Calibration: solved data drops, updated input string to new format #1473

Merged
iravasen merged 3 commits into
AliceO2Group:masterfrom
IsakovAD:IsakovAD_Calibration_NewInputString
Sep 29, 2022
Merged

ITS Online Calibration: solved data drops, updated input string to new format #1473
iravasen merged 3 commits into
AliceO2Group:masterfrom
IsakovAD:IsakovAD_Calibration_NewInputString

Conversation

@IsakovAD
Copy link
Copy Markdown
Collaborator

@IsakovAD IsakovAD commented Sep 23, 2022

  • Fixed data drops (by Ivan Ravasenga)
  • New format of input string

@IsakovAD IsakovAD requested a review from iravasen as a code owner September 23, 2022 15:23
Comment thread Modules/ITS/src/ITSThresholdCalibrationTask.cxx Outdated
Comment thread Modules/ITS/src/ITSThresholdCalibrationTask.cxx Outdated
@IsakovAD
Copy link
Copy Markdown
Collaborator Author

@iravasen all comments were applied in the new commit

@iravasen
Copy link
Copy Markdown
Collaborator

Thanks @IsakovAD, approving (cc @knopers8 )

@iravasen iravasen changed the title [WIP] ITS Online Calibration: solved data drops, updated input string to new format ITS Online Calibration: solved data drops, updated input string to new format Sep 27, 2022
@iravasen iravasen enabled auto-merge (squash) September 27, 2022 11:59
@iravasen
Copy link
Copy Markdown
Collaborator

The error is not related to this PR. @Barthelemy or @knopers8 can you please force merge and then merge also AliceO2Group/ControlWorkflows#509 which is directly linked to this PR? Thank you!

@iravasen iravasen merged commit bcd5158 into AliceO2Group:master Sep 29, 2022
knopers8 pushed a commit that referenced this pull request Oct 3, 2022
…w format (#1473)

* Solved data drops, updated to new data string format

* Ivan's comments fixed

* Clang
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