Skip to content

Conversation

@njacazio
Copy link
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
Collaborator Author

@jesgum this works

@github-actions
Copy link

github-actions bot commented Nov 24, 2025

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

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 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
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
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