Skip to content

MCH: add bad digits time errors#1157

Merged
Barthelemy merged 5 commits intoAliceO2Group:masterfrom
AliceMCH:mch-add-bad-digits-time-error
Apr 27, 2022
Merged

MCH: add bad digits time errors#1157
Barthelemy merged 5 commits intoAliceO2Group:masterfrom
AliceMCH:mch-add-bad-digits-time-error

Conversation

@aferrero2707
Copy link
Contributor

The digit time errors are added to existing QC histograms.
This is the follow-up of AliceO2Group/AliceO2#8573.

In addition, this PR introduces some further visual improvements to several plots, without changing the actual contents of the plots or the way they are filled.

The histograms are now plotted without gaps between the chambers,
and the chamber label has been moved to the bottom of the plot for
better visibility
@aferrero2707
Copy link
Contributor Author

@Barthelemy @knopers8 it would you be available to review this PR such that it could be included in the next FLP suite?
Thanks!

@Barthelemy
Copy link
Collaborator

@aferrero2707 yes

@TimoWilken There seems to be an issue with the mac builder (?)

@aferrero2707
Copy link
Contributor Author

The issues in the macOS build are similar to those occurring in #1144

@Barthelemy Barthelemy merged commit 7486ee7 into AliceO2Group:master Apr 27, 2022
@aferrero2707 aferrero2707 deleted the mch-add-bad-digits-time-error branch June 20, 2022 20:53
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