Skip to content

[ALICE3] only load lut if requested and move lut cleanup to geo provider #14910

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 6 commits into from
Feb 11, 2026
Merged

[ALICE3] only load lut if requested and move lut cleanup to geo provider #14910

njacazio merged 6 commits into from
Feb 11, 2026
+117 −106

Conversation

@jesgum
Copy link
Contributor

@jesgum jesgum commented Feb 11, 2026

  • Only load luts for rich and tof if requested (until if we manage to load them shared between the tasks)
  • Move the cleanLutWhenLoaded configurable to the geometry provider to avoid duplicating this setting for all tasks that needs the luts

@njacazio

@github-actions github-actions bot changed the title only load lut if requested and move lut cleanup to geo provider [ALICE3] only load lut if requested and move lut cleanup to geo provider Feb 11, 2026
@jesgum jesgum marked this pull request as ready for review February 11, 2026 10:54
@jesgum jesgum marked this pull request as draft February 11, 2026 11:35
@jesgum jesgum marked this pull request as ready for review February 11, 2026 11:45
njacazio
njacazio previously approved these changes Feb 11, 2026
View reviewed changes
@njacazio njacazio enabled auto-merge (squash) February 11, 2026 11:54
@alibuild
Copy link
Collaborator

alibuild commented Feb 11, 2026

Error while checking build/O2Physics/o2 for 281b4bd at 2026-02-11 13:35:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14910-slc9_x86-64/0/ALICE3/TableProducer/OTF/onTheFlyTracker.cxx:121:387: error: redeclaration of 'o2::framework::Configurable<std::__cxx11::basic_string<char> > OnTheFlyTracker::primaryVertexOption'
ninja: build stopped: subcommand failed.

Full log here.

auto-merge was automatically disabled February 11, 2026 13:18

Head branch was pushed to by a user without write access

@njacazio njacazio merged commit 8cb945f into AliceO2Group:master Feb 11, 2026
19 checks passed
@jesgum jesgum deleted the lut-loading branch February 11, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@njacazio njacazio njacazio approved these changes

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

@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.

3 participants

@jesgum @alibuild @njacazio