Skip to content

Avoid exposing rANS internal details to ROOT#11892

Merged
TimoWilken merged 1 commit into
AliceO2Group:devfrom
ktf:avoid-root-rANS
Sep 14, 2023
Merged

Avoid exposing rANS internal details to ROOT#11892
TimoWilken merged 1 commit into
AliceO2Group:devfrom
ktf:avoid-root-rANS

Conversation

@ktf
Copy link
Copy Markdown
Member

@ktf ktf commented Sep 14, 2023

No description provided.

@ktf ktf requested a review from shahor02 as a code owner September 14, 2023 09:11
@ktf
Copy link
Copy Markdown
Member Author

ktf commented Sep 14, 2023

This is probably the quickest way to fix it.

@davidrohr
Copy link
Copy Markdown
Collaborator

davidrohr commented Sep 14, 2023 via email

@ktf
Copy link
Copy Markdown
Member Author

ktf commented Sep 14, 2023

@TimoWilken could you check how come the fullsystem test takes so long?

@TimoWilken
Copy link
Copy Markdown
Contributor

@ktf Uploading the logs failed because they weren't valid UTF-8 apparently. I've just fixed that in ali-bot and set the fullCI status manually for this PR. The o2-cs8 check is still running, but almost done.

@TimoWilken TimoWilken merged commit ab9b395 into AliceO2Group:dev Sep 14, 2023
@TimoWilken
Copy link
Copy Markdown
Contributor

Merged since all of the checks were green (except for the Intel Mac one, but that's a known problem and the compilation succeeded, at least).

@ktf ktf deleted the avoid-root-rANS branch September 14, 2023 17:35
leo-barreto pushed a commit to leo-barreto/AliceO2 that referenced this pull request Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants