Skip to content

Conversation

@glromane
Copy link
Contributor

@glromane glromane commented Jan 21, 2025

With this PR I would like to access the list of tables also when analysing data without metadata (e.g. derived data).
@ktf I don't know if there are repercussions for this change that I did not foresee but my tests on the analysis workflow were successful.
Tagging also @ddobrigk
Cheers,
Gleb
feat @njacazio

@github-actions
Copy link
Contributor

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

@glromane
Copy link
Contributor Author

Example of usage: AliceO2Group/O2Physics#9438

@ktf
Copy link
Member

ktf commented Jan 23, 2025

It makes sense. Can you switch to non-Draft so that we can check?

@glromane glromane marked this pull request as ready for review January 23, 2025 12:00
@glromane glromane requested a review from a team as a code owner January 23, 2025 12:00
@njacazio
Copy link
Collaborator

Hi @ktf, I think this is Gleb's first PR in O2 and needs to be approved for the tests

@njacazio
Copy link
Collaborator

On top of the local tests also CI is ok @ktf

@ktf ktf merged commit 0d3b460 into AliceO2Group:dev Jan 24, 2025
14 checks passed
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