Skip to content

[EMCAL-752] Fix number of LEDMON channel in Pedestal object#12959

Merged
mfasDa merged 1 commit intoAliceO2Group:devfrom
jokonig:EMCAL-752
Apr 2, 2024
Merged

[EMCAL-752] Fix number of LEDMON channel in Pedestal object#12959
mfasDa merged 1 commit intoAliceO2Group:devfrom
jokonig:EMCAL-752

Conversation

@jokonig
Copy link
Collaborator

@jokonig jokonig commented Mar 29, 2024

  • In total, only 480 LEDMON channels are in the EMCal (was 17664 in the code)
  • Subsequently also modified the plotting of the 2D map of the object

@jokonig jokonig requested a review from mfasDa as a code owner March 29, 2024 13:51
@github-actions
Copy link
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-apass
async-2023-pp-apass1
async-2022-pp-apass6
async-2022-pp-apass4
async-mc
async-data

- In total, only 480 LEDMON channels are in the EMCal (was 17664 in the code)
- Subsequently also modified the plotting of the 2D map of the object
Copy link
Collaborator

@mfasDa mfasDa 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!

@mfasDa mfasDa merged commit 13dccf6 into AliceO2Group:dev Apr 2, 2024
andreasmolander pushed a commit to andreasmolander/AliceO2 that referenced this pull request Apr 12, 2024
…roup#12959)

- In total, only 480 LEDMON channels are in the EMCal (was 17664 in the code)
- Subsequently also modified the plotting of the 2D map of the object

Co-authored-by: jokonig <jokonig@cern.ch>
andreasmolander pushed a commit to andreasmolander/AliceO2 that referenced this pull request Apr 12, 2024
…roup#12959)

- In total, only 480 LEDMON channels are in the EMCal (was 17664 in the code)
- Subsequently also modified the plotting of the 2D map of the object

Co-authored-by: jokonig <jokonig@cern.ch>
mwinn2 pushed a commit to mwinn2/AliceO2 that referenced this pull request Apr 25, 2024
…roup#12959)

- In total, only 480 LEDMON channels are in the EMCal (was 17664 in the code)
- Subsequently also modified the plotting of the 2D map of the object

Co-authored-by: jokonig <jokonig@cern.ch>
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