Skip to content

Conversation

@jikim1290
Copy link
Contributor

adding process with shifted vectors

@github-actions
Copy link

github-actions bot commented Nov 10, 2025

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

Please consider the following formatting changes to AliceO2Group#13758
}
}
}
PROCESS_SWITCH(jEPFlowAnalysis, process, "default process", true);
Copy link
Collaborator

Choose a reason for hiding this comment

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

I guess this is not needed for process
I doubt it can be disabled

if (std::abs(coll.posZ()) > cfgVertexZ)
return;
switch (cfgEvtSel) {
case 0: // Sel8
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would suggest using named constants

@victor-gonzalez victor-gonzalez enabled auto-merge (squash) November 10, 2025 20:06
@victor-gonzalez victor-gonzalez merged commit 7a6c1c2 into AliceO2Group:master Nov 10, 2025
14 of 15 checks passed
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
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.

3 participants