Skip to content

[Common] Fix issue 14173#16814

Open
amaringarcia wants to merge 3 commits into
AliceO2Group:masterfrom
amaringarcia:master
Open

[Common] Fix issue 14173#16814
amaringarcia wants to merge 3 commits into
AliceO2Group:masterfrom
amaringarcia:master

Conversation

@amaringarcia

@amaringarcia amaringarcia commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Fixes #14173

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Fixing issues/14173 [Common] Fixing issues/14173 Jun 25, 2026
@vkucera

vkucera commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Thanks a lot @amaringarcia !

@vkucera

vkucera commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

It seems the identicalConditionAfterEarlyExit warnings are still there.

auto& vecOccPrimUnfm80 = occPrimUnfm80[i];
float meanOccPrimUnfm80 = TMath::Mean(vecOccPrimUnfm80.size(), vecOccPrimUnfm80.data());
normalizeVector(vecOccPrimUnfm80, meanOccPrimUnfm80 / meanOccPrimUnfm80);
//normalizeVector(vecOccPrimUnfm80, meanOccPrimUnfm80 / meanOccPrimUnfm80);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there a reason to keep it?

@vkucera vkucera changed the title [Common] Fixing issues/14173 [Common] Fix issue 14173 Jun 25, 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.

Potential bugs in occupancyTableProducer.cxx

2 participants