Skip to content

Conversation

@alcaliva
Copy link
Collaborator

@alcaliva alcaliva commented Sep 9, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Sep 9, 2025

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

@alcaliva
Copy link
Collaborator Author

Hi @ktf,
the checks on build/O2Physics/o2/macOS and build/O2Physics/o2/macOS-arm keep failing since yesterday. Do you perhaps know what is the issue? Thanks a lot!
Alberto

@ktf
Copy link
Member

ktf commented Sep 10, 2025

@singiamtel is looking into it

@f3sch
Copy link
Contributor

f3sch commented Sep 10, 2025

@alcaliva, there is also some very basic task we used in the past to validate the thinning procedure, which may could be used for this validation here, may need some adaptions for the MC tables though

@alcaliva
Copy link
Collaborator Author

Hi @f3sch,
thanks! We might also use that. I think that this code is sufficient to test randomness in the reconstruction software. If the pt distributions of the same events are exactly equal, then we can use this as a validation tool.

@f3sch
Copy link
Contributor

f3sch commented Sep 10, 2025

indeed. Anyways, I think but I might be very well mistaken, even on cpu the tpc reco is not deterministic when using multiple threads, so a 1:1 mapping might not be possible.

@alcaliva
Copy link
Collaborator Author

Still, I think that quantifying the deviations due to this effect would be good.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for bfe34e8 at 2025-09-11 23:55:

No log files found

Full log here.

@alcaliva
Copy link
Collaborator Author

Hi, can this PR be approved? the failure of o2/macOS is not caused by this

@ktf
Copy link
Member

ktf commented Sep 19, 2025

@singiamtel , I think you need to cleanup also this builder. It's probably the stale branch.

@alcaliva alcaliva merged commit c2db361 into AliceO2Group:master Oct 4, 2025
11 of 12 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Oct 11, 2025
ArkaprabhaSaha001 pushed a commit to ArkaprabhaSaha001/O2Physics that referenced this pull request Oct 21, 2025
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 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.

5 participants