Skip to content

[ALICE3] Fix LUT file paths and whitespace handling - #13953

Merged
njacazio merged 2 commits into
masterfrom
njacazio-patch-5
Nov 24, 2025
Merged

[ALICE3] Fix LUT file paths and whitespace handling#13953
njacazio merged 2 commits into
masterfrom
njacazio-patch-5

Conversation

@njacazio

Copy link
Copy Markdown
Collaborator

Updated LUT file paths to remove trailing spaces and ensure proper loading.

Updated LUT file paths to remove trailing spaces and ensure proper loading.
@njacazio

Copy link
Copy Markdown
Collaborator Author

@jesgum this works

@github-actions

github-actions Bot commented Nov 24, 2025

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Fix LUT file paths and whitespace handling [ALICE3] Fix LUT file paths and whitespace handling Nov 24, 2025
Add check for empty LUT file and log a fatal error if it is empty.
@njacazio
njacazio marked this pull request as ready for review November 24, 2025 16:36
@njacazio
njacazio enabled auto-merge (squash) November 24, 2025 16:36
@njacazio
njacazio disabled auto-merge November 24, 2025 16:36
@njacazio
njacazio enabled auto-merge (squash) November 24, 2025 16:37

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

@njacazio
njacazio merged commit aa198e0 into master Nov 24, 2025
14 of 16 checks passed
@njacazio
njacazio deleted the njacazio-patch-5 branch November 24, 2025 21:36
yakparo pushed a commit to yakparo/O2Physics that referenced this pull request Nov 29, 2025
mapalhares pushed a commit to mapalhares/O2Physics that referenced this pull request Jan 20, 2026
fcolamar pushed a commit to fcolamar/O2Physics that referenced this pull request Feb 4, 2026
karlazertova pushed a commit to karlazertova/O2Physics that referenced this pull request Feb 5, 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.

2 participants