Skip to content

Conversation

@jokonig
Copy link
Collaborator

@jokonig jokonig commented Jul 1, 2025

  • Gain of EMCal cell is temperature dependent.
  • The temperature is stored in the ccdb about every 3min for 8 sensors per SM
  • New class downloads temperature values from ccdb and averages them per SM
  • Per cell, the energy dependence was determined in run2. These values are stored in the ccdb and also read in at initializeFromCCDB
  • As the temperature values are accumulated per run, the end of run timestamp is taken - 60s of safety margin

- Gain of EMCal cell is temperature dependent.
- The temperature is stored in the ccdb about every 3min for 8 sensors per SM
- New class downloads temperature values from ccdb and averages them per SM
- Per cell, the energy dependence was determined in run2. These values are stored in the ccdb and also read in at initializeFromCCDB
- As the temperature values are accumulated per run, the end of run timestamp is taken - 60s of safety margin
@jokonig jokonig requested a review from mfasDa as a code owner July 1, 2025 18:35
@github-actions
Copy link
Contributor

github-actions bot commented Jul 1, 2025

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-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@jokonig jokonig requested review from aferrero2707, ktf and sawenzel July 1, 2025 19:47
@jokonig
Copy link
Collaborator Author

jokonig commented Jul 2, 2025

@ktf @sawenzel I believe the error in the aarch64 build is unrelated to this PR, right?
If yes, this PR could be merged :)

@ktf
Copy link
Member

ktf commented Jul 2, 2025

It looks like a spurious error from QC.

@jokonig
Copy link
Collaborator Author

jokonig commented Jul 2, 2025

@ktf could you then approve this PR :)

Copy link
Collaborator

@nstrangm nstrangm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks!

@sawenzel sawenzel merged commit ad44565 into AliceO2Group:dev Jul 3, 2025
13 checks passed
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.

4 participants