Skip to content

[ALICE3] A3 OTF tracker, simplify histo filling #14333

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
njacazio merged 5 commits into from
Dec 24, 2025
Merged

[ALICE3] A3 OTF tracker, simplify histo filling #14333

njacazio merged 5 commits into from
Dec 24, 2025
+166 −163

Conversation

@njacazio
Copy link
Collaborator

@njacazio njacazio commented Dec 23, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Dec 23, 2025 •
edited
Loading

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

@njacazio
Copy link
Collaborator Author

njacazio commented Dec 23, 2025

@jesgum

@github-actions github-actions bot changed the title A3 OTF tracker, simplify histo filling [ALICE3] A3 OTF tracker, simplify histo filling Dec 23, 2025
@njacazio njacazio marked this pull request as ready for review December 23, 2025 08:42
@njacazio njacazio enabled auto-merge (squash) December 23, 2025 08:42
alibuild
alibuild previously approved these changes Dec 23, 2025
View reviewed changes
Copy link
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 @njacazio.

@alibuild
Copy link
Collaborator

alibuild commented Dec 23, 2025

Error while checking build/O2Physics/o2 for 38b2e67 at 2025-12-23 10:48:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14333-slc9_x86-64/0/ALICE3/TableProducer/OTF/onTheFlyTracker.cxx:834:55: error: 'mGeoContainer' was not declared in this scope
ninja: build stopped: subcommand failed.

Full log here.

@njacazio njacazio disabled auto-merge December 24, 2025 09:48
@njacazio njacazio enabled auto-merge (squash) December 24, 2025 09:48
alibuild
alibuild previously approved these changes Dec 24, 2025
View reviewed changes
Copy link
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 @njacazio.

@njacazio njacazio disabled auto-merge December 24, 2025 12:06
@njacazio njacazio enabled auto-merge (squash) December 24, 2025 12:06
alibuild
alibuild approved these changes Dec 24, 2025
View reviewed changes
Copy link
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 @njacazio.

@njacazio njacazio merged commit b123a95 into AliceO2Group:master Dec 24, 2025
11 of 12 checks passed
@njacazio njacazio deleted the nj-a3configuration branch December 24, 2025 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild alibuild approved these changes

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

Assignees

No one assigned

Labels

alice3

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@njacazio @alibuild