Skip to content

[EMCAL-524] Plots for different trigger classes#590

Merged
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
mfasDa:EMCAL-524
Jan 15, 2021
Merged

[EMCAL-524] Plots for different trigger classes#590
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
mfasDa:EMCAL-524

Conversation

@mfasDa
Copy link
Copy Markdown
Collaborator

@mfasDa mfasDa commented Jan 14, 2021

  • Create and fill QA plots for Calib and Physics Triggers in Raw task
  • ADC range of raw amplitude projection increased in order to cover the
    range of LED events

@mfasDa
Copy link
Copy Markdown
Collaborator Author

mfasDa commented Jan 14, 2021

@Barthelemy This is the commit in the raw task which does not depend on changes in O2. It should be handled with high priority

- Create and fill QA plots for Calib and Physics Triggers in Raw task
- ADC range of raw amplitude projection increased in order to cover the
  range of LED events
@Barthelemy
Copy link
Copy Markdown
Collaborator

@TimoWilken would it be possible to know where in the queue this PR is with regards to the macOS build ?

@TimoWilken
Copy link
Copy Markdown
Contributor

@Barthelemy it's building now. There was still a problem with some of the large git repos on the CI server, I've disabled git gc for now, so it doesn't keep repacking them.

@mfasDa
Copy link
Copy Markdown
Collaborator Author

mfasDa commented Jan 14, 2021

Build error unrelated

@Barthelemy Barthelemy merged commit a5f0049 into AliceO2Group:master Jan 15, 2021
knopers8 pushed a commit to knopers8/QualityControl that referenced this pull request Jan 23, 2021
- Create and fill QA plots for Calib and Physics Triggers in Raw task
- ADC range of raw amplitude projection increased in order to cover the
  range of LED events

Co-authored-by: Cristina Terrevoli <cristina.terrevoli@cern.ch>
@mfasDa mfasDa deleted the EMCAL-524 branch January 25, 2021 10:03
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