Skip to content

[PWGEM] Revert "[PWGEM] LMee: Adding more DCA correlation histograms to various signals."#15734

Merged
hscheid merged 1 commit intomasterfrom
revert-15602-master
Apr 13, 2026
Merged

[PWGEM] Revert "[PWGEM] LMee: Adding more DCA correlation histograms to various signals."#15734
hscheid merged 1 commit intomasterfrom
revert-15602-master

Conversation

@hscheid
Copy link
Copy Markdown
Collaborator

@hscheid hscheid commented Apr 12, 2026

Reverts #15602

The above PR seems to break the MC analysis as the beauty histograms are defined twice:
Error from the HY test

[ERROR] SEVERE: Device dielectron-mc (561166) had at least one message above severity 7: Hash collision in HistogramRegistry "output"! Please rename histogram "Pair/bbbar/b2l_b2l/uls/hDCAz1vsDCAz2" or "Pair/bbbar/b2l_b2l/uls/hDCAz1vsDCAz2".

This can be traced back to
This defines the histogram for charm:

fRegistry.add(std::format("Pair/ccbar/c2l_c2l/{0}hDCAz1vsDCAz2", strSign), "DCA_{z,1} vs DCA_{z,2}", o2::framework::kTH2F, {axis_dca_track1, axis_dca_track2}, true);

Then here the charm histograms are copied to the beauty histograms:

fRegistry.addClone("Pair/ccbar/c2l_c2l/", "Pair/bbbar/b2l_b2l/");

and here it is defined again:

fRegistry.add(std::format("Pair/bbbar/b2l_b2l/{0}hDCAz1vsDCAz2", strSign), "DCA_{z,1} vs DCA_{z,2}", o2::framework::kTH2F, {axis_dca_track1, axis_dca_track2}, true);

@github-actions github-actions bot added the pwgem label Apr 12, 2026
@github-actions github-actions bot changed the title Revert "[PWGEM] LMee: Adding more DCA correlation histograms to various signals." [PWGEM] Revert "[PWGEM] LMee: Adding more DCA correlation histograms to various signals." Apr 12, 2026
@github-actions
Copy link
Copy Markdown

O2 linter results: ❌ 263 errors, ⚠️ 241 warnings, 🔕 0 disabled

@hscheid hscheid enabled auto-merge (squash) April 13, 2026 07:40
Copy link
Copy Markdown
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @hscheid.

@hscheid hscheid merged commit e93ccf1 into master Apr 13, 2026
13 of 16 checks passed
@hscheid hscheid deleted the revert-15602-master branch April 13, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants