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

Common: support varibale bins in reference comparator output #2440

Conversation

aferrero2707
Copy link
Contributor

@aferrero2707 aferrero2707 commented Sep 25, 2024

The changes allow to properly display histograms with variable bin sizes in the output of the reference comparator task.

Here is an example of variable bin size histogram compared to reference:
image

@aferrero2707
Copy link
Contributor Author

@Barthelemy @knopers8 @justonedev1 again a requests from detector experts, only requiring rather trivial changes.

@knopers8 knopers8 enabled auto-merge (squash) September 25, 2024 10:10
@aferrero2707 aferrero2707 changed the title Common: support varibale bins in reference comparator output [WIP] Common: support varibale bins in reference comparator output Sep 25, 2024
auto-merge was automatically disabled September 25, 2024 17:07

Head branch was pushed to by a user without write access

@aferrero2707 aferrero2707 force-pushed the common-support-variable-bin-size-in-reference-comparator branch from 1c9f72b to b259543 Compare September 25, 2024 17:07
@aferrero2707 aferrero2707 changed the title [WIP] Common: support varibale bins in reference comparator output Common: support varibale bins in reference comparator output Sep 25, 2024
@aferrero2707
Copy link
Contributor Author

@knopers8 I had to change a bit the code, because the use of the Clone() function was giving empty plots when the source histograms were of THRatio type. Not clear why, but anyhow the updated code is also quite trivial.

The changes allow to properly display histograms with variable bin
sizes in the output of the reference comparator task.
@aferrero2707 aferrero2707 force-pushed the common-support-variable-bin-size-in-reference-comparator branch from b259543 to 7aef72b Compare September 25, 2024 17:14
@Barthelemy Barthelemy merged commit 779dda4 into AliceO2Group:master Sep 26, 2024
7 checks passed
@aferrero2707 aferrero2707 deleted the common-support-variable-bin-size-in-reference-comparator branch September 26, 2024 07:06
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.

3 participants