Skip to content

[PWGLF] Fix bug#16309

Merged
romainschotter merged 2 commits into
masterfrom
romainschotter-patch-1
May 19, 2026
Merged

[PWGLF] Fix bug#16309
romainschotter merged 2 commits into
masterfrom
romainschotter-patch-1

Conversation

@romainschotter
Copy link
Copy Markdown
Collaborator

@romainschotter romainschotter commented May 19, 2026 •
edited
Loading

@lorber98 @BanajitBarman for your information

@github-actions github-actions Bot added the pwglf label May 19, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026 •
edited
Loading

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

@github-actions github-actions Bot changed the title Fix bug [PWGLF] Fix bug May 19, 2026
@romainschotter romainschotter marked this pull request as ready for review May 19, 2026 15:47
@romainschotter romainschotter enabled auto-merge (squash) May 19, 2026 15:47
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented May 19, 2026

Error while checking build/O2Physics/staging for 0909d86 at 2026-05-19 17:53:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/TableProducer/Common/mcCentrality.cxx:146:27: error: 'this' pointer is null [-Werror=nonnull]
ninja: build stopped: subcommand failed.

Full log here.

omvazque
omvazque previously approved these changes May 19, 2026
View reviewed changes
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented May 19, 2026

Error while checking build/O2Physics/o2 for 0909d86 at 2026-05-19 17:57:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/TableProducer/Common/mcCentrality.cxx:146:27: error: 'this' pointer is null [-Werror=nonnull]
ninja: build stopped: subcommand failed.

Full log here.

@romainschotter
Remove SetDirectory call on histogram
cbebc3b 
Removed the call to SetDirectory(0) on the histogram object.
@romainschotter romainschotter merged commit 1767dc1 into master May 19, 2026
14 of 15 checks passed
@romainschotter romainschotter deleted the romainschotter-patch-1 branch May 19, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mpuccio mpuccio mpuccio approved these changes

@omvazque omvazque omvazque left review comments

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@skundu692 skundu692 Awaiting requested review from skundu692 skundu692 is a code owner

Assignees

No one assigned

Labels

pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@romainschotter @alibuild @mpuccio @omvazque