Skip to content

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

Merged
njacazio merged 6 commits intoAliceO2Group:masterfrom
jesgum:lut-loading
Feb 11, 2026
Merged

[ALICE3] only load lut if requested and move lut cleanup to geo provider#14910
njacazio merged 6 commits intoAliceO2Group:masterfrom
jesgum:lut-loading

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
@njacazio njacazio enabled auto-merge (squash) February 11, 2026 11:54
@alibuild
Copy link
Collaborator

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
comrademarvin pushed a commit to comrademarvin/O2Physics that referenced this pull request Mar 16, 2026
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