Skip to content

[PWGLF] updated the config axis with variable width #14643

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.

Sign up for GitHub

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

Jump to bottom
Merged
romainschotter merged 1 commit into from
Jan 26, 2026
Merged

[PWGLF] updated the config axis with variable width #14643

romainschotter merged 1 commit into from
Jan 26, 2026

Conversation

@prottayCMT
Copy link
Contributor

@prottayCMT prottayCMT commented Jan 26, 2026

No description provided.

@github-actions github-actions bot added the pwglf label Jan 26, 2026
@github-actions
Copy link

github-actions bot commented Jan 26, 2026

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

@github-actions github-actions bot changed the title updated the config axis with variable width [PWGLF] updated the config axis with variable width Jan 26, 2026
@romainschotter
Copy link
Collaborator

romainschotter commented Jan 26, 2026

Hi @prottayCMT !
When you have some time, can you please fix the Use constant references for non-modified iterators in range-based for loops. warnings? It is a small thing which allows to optimize the code performance :-)
Thank you very much!

@romainschotter romainschotter enabled auto-merge (squash) January 26, 2026 17:22
@romainschotter romainschotter merged commit 2854f73 into AliceO2Group:master Jan 26, 2026
11 of 13 checks passed
@prottayCMT
Copy link
Contributor Author

prottayCMT commented Jan 26, 2026

Hi @prottayCMT ! When you have some time, can you please fix the Use constant references for non-modified iterators in range-based for loops. warnings? It is a small thing which allows to optimize the code performance :-) Thank you very much!

Thanks Romain....Next time I will keep this in mind to change before PR approval.

monamelop pushed a commit to monamelop/O2Physics that referenced this pull request Jan 28, 2026
@prottayCMT @monamelop
[PWGLF] updated the config axis with variable width (AliceO2Group#14643)
d146cf6 
Co-authored-by: Prottay Das <prottay@alipap1.cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@romainschotter romainschotter romainschotter approved these changes

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

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

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

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

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

Assignees

No one assigned

Labels

pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@prottayCMT @romainschotter