Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please consider the following formatting changes to #11974 #86

Closed

Conversation

alibuild
Copy link

This PR cannot be merged as is. You should either run clang-format yourself and update the pull request, or merge this PR in yours.
You can find the AliceO2 coding conventions at https://github.com/AliceO2Group/CodingGuidelines.

@alibuild alibuild closed this Sep 29, 2023
@alibuild alibuild deleted the alibot-cleanup-11974 branch September 29, 2023 11:45
chiarazampolli pushed a commit that referenced this pull request Dec 16, 2023
* Update reconstruction objects (#83)

* Update reconstruction objects

* Update ZDCTowerParam.cxx

* Tune intercalib (#84)

* Small update to intercalibration

* Small update to intercalibration

* Please consider the following formatting changes (#81)

---------

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>

* Tuning of intercalibration - WIP

* - corrected centroid computation for ZP calorimeters
- rms of ZP centroid
- updates in Intercalibration
- introduction of offsets for energy and amplitud calibration

* Please consider the following formatting changes (#85)

* Modify offset for channels that are directly calibrated

* Fix bug

* ADC and TDC offsets

* Please consider the following formatting changes (#86)

* Fixes

* Amplitude cannot be calibrated in GeV. Use RecEventFlat to set unit of measurement

* Fixes

* Fixes

* Fixes

* Fixes

* Please consider the following formatting changes (#87)

---------

Co-authored-by: ALICE Builder <alibuild@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants