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
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Apr 27, 2026
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