Skip to content

Conversation

@fgrosa
Copy link
Contributor

@fgrosa fgrosa commented Sep 24, 2025

Hi @catalinristea this PR adds the D0 meson in the QC. I however noticed that the configuration for the other workflows is pretty old, how old is the O2Physics tag used in the QC? If very old, we might need to readjust a few configurations for the D0. Thanks!

@github-actions
Copy link

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@sawenzel
Copy link
Contributor

@fgrosa : It should be easy to enforce latest O2Physics via the 2tag approach. We simply need to mark the analysis QC tasks in this direction. I think this makes most sense .. given that also hyperloop operates with latest dailies.

@fgrosa
Copy link
Contributor Author

fgrosa commented Sep 24, 2025

@fgrosa : It should be easy to enforce latest O2Physics via the 2tag approach. We simply need to mark the analysis QC tasks in this direction. I think this makes most sense .. given that also hyperloop operates with latest dailies.

Hi @sawenzel indeed I agree with you! I just noticed that the other instances have all dependences that are currently deprecated (they are still in the latest O2Physics tags, but it would be good to move to the new ones).
Also please note that this is mostly for data rather than MC, so I am not sure if the 2tag approach can also be used for QC of data reconstruction.

@catalinristea
Copy link
Collaborator

Hi!
PR was tested and found fine for async reco Pb-Pb. It can be merged and put it in production.
Thanks a lot, Fabrizio!

Yours, Catalin.

@sawenzel sawenzel merged commit f8f369a into AliceO2Group:master Sep 26, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants