Skip to content

[ALICE3] Add LUT utilities - #17546

Open
njacazio wants to merge 12 commits into
masterfrom
njacazio-patch-3
Open

[ALICE3] Add LUT utilities#17546
njacazio wants to merge 12 commits into
masterfrom
njacazio-patch-3

Conversation

@njacazio

@njacazio njacazio commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Aug 21, 2026 •
edited
Loading

Copy link
Copy Markdown

O2 linter results: ❌ 1 errors, ⚠️ 35 warnings, 🔕 0 disabled

@github-actions github-actions Bot changed the title Refactor lutEntry_t print method and size calculations [ALICE3] Refactor lutEntry_t print method and size calculations Aug 21, 2026
@njacazio
Implement getEntryIndex method in FlatLutEntry
059b341 
Add method to compute linear index for LUT entries
@njacazio
Add files via upload
b0640be
@njacazio njacazio changed the title [ALICE3] Refactor lutEntry_t print method and size calculations [ALICE3] Add LUT utilities Aug 21, 2026
@alibuild alibuild mentioned this pull request Aug 21, 2026
Merged
@njacazio
njacazio marked this pull request as ready for review August 21, 2026 13:59
@alibuild alibuild mentioned this pull request Aug 21, 2026
Merged
@njacazio
njacazio enabled auto-merge (squash) August 21, 2026 14:36
alibuild
alibuild previously approved these changes Aug 21, 2026
View reviewed changes

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

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

alibuild commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 7b9b498 at 2026-08-21 16:51:

No log files found

Full log here.

@alibuild

alibuild commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 7b9b498 at 2026-08-21 16:51:

No log files found

Full log here.

@alibuild alibuild mentioned this pull request Aug 21, 2026
Merged
@alibuild

alibuild commented Aug 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for c9b194c at 2026-08-21 19:57:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:212:17: error: 'previewHeader' was not declared in this scope; did you mean 'PreviewHeader'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:225:13: error: no declaration matches 'o2::delphes::lutHeader_t o2::delphes::FlatLutData::previewHeader(const uint8_t*, size_t)'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:268:13: error: no declaration matches 'o2::delphes::lutHeader_t o2::delphes::FlatLutData::previewHeader(std::ifstream&, const char*)'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:284:28: error: 'previewHeader' was not declared in this scope; did you mean 'PreviewHeader'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatTrackSmearer.cxx:143:32: error: 'previewHeader' is not a member of 'o2::delphes::FlatLutData'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatTrackSmearer.cxx:166:32: error: 'previewHeader' is not a member of 'o2::delphes::FlatLutData'
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Aug 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for c9b194c at 2026-08-21 18:21:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:212:17: error: 'previewHeader' was not declared in this scope; did you mean 'PreviewHeader'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:225:13: error: no declaration matches 'o2::delphes::lutHeader_t o2::delphes::FlatLutData::previewHeader(const uint8_t*, size_t)'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:268:13: error: no declaration matches 'o2::delphes::lutHeader_t o2::delphes::FlatLutData::previewHeader(std::ifstream&, const char*)'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:284:28: error: 'previewHeader' was not declared in this scope; did you mean 'PreviewHeader'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatTrackSmearer.cxx:143:32: error: 'previewHeader' is not a member of 'o2::delphes::FlatLutData'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatTrackSmearer.cxx:166:32: error: 'previewHeader' is not a member of 'o2::delphes::FlatLutData'
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Aug 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for c9b194c at 2026-08-21 18:10:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:212:17: error: 'previewHeader' was not declared in this scope; did you mean 'PreviewHeader'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:225:13: error: no declaration matches 'o2::delphes::lutHeader_t o2::delphes::FlatLutData::previewHeader(const uint8_t*, size_t)'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:268:13: error: no declaration matches 'o2::delphes::lutHeader_t o2::delphes::FlatLutData::previewHeader(std::ifstream&, const char*)'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatLutEntry.cxx:284:28: error: 'previewHeader' was not declared in this scope; did you mean 'PreviewHeader'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatTrackSmearer.cxx:143:32: error: 'previewHeader' is not a member of 'o2::delphes::FlatLutData'
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/ALICE3/Core/FlatTrackSmearer.cxx:166:32: error: 'previewHeader' is not a member of 'o2::delphes::FlatLutData'
ninja: build stopped: subcommand failed.

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild alibuild left review comments
@hscheid hscheid Awaiting requested review from hscheid hscheid is a code owner
@jesgum jesgum Awaiting requested review from jesgum jesgum is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

alice3

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@njacazio @alibuild