Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ntuple] Add RNTupleProcessor::PrepareJoinModel #17882

Closed

Conversation

enirolf
Copy link
Contributor

@enirolf enirolf commented Mar 4, 2025

The aim is to change the vector of model that currently needs to be passed to RNTupleProcessor::CreateJoin to a single model that already represents what the joined entries should look like. From the user's point of view, creating such models from scratch is tricky because we have to do some restructuring to prevent field name clashes. This static helper method provides a way to (correctly) create join models as expected by the processor.

To avoid having to freeze models and access all fields through an REntry, RNTupleModel::GetFieldNames is added as a companion to the Get[Const|Mutable]Field methods.

enirolf added 2 commits March 4, 2025 18:01
The aim is to change the vector of model that currently needs to be
passed to `RNTupleProcessor::CreateJoin` to a single model that already
represents what the joined entries should look like. From the user's
point of view, creating such models from scratch is tricky because we
have to do some resturcturing to prevent field name clashes. This helper
method provides a way to (correctly) create join models as expected by
the processor.
@enirolf enirolf self-assigned this Mar 4, 2025
Copy link

github-actions bot commented Mar 4, 2025

Test Results

    18 files      18 suites   4d 11h 55m 46s ⏱️
 2 724 tests  2 723 ✅ 0 💤 1 ❌
47 341 runs  47 340 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit c37e563.

@enirolf enirolf closed this Mar 5, 2025
@enirolf enirolf deleted the ntuple-processor-prep-join-model branch March 5, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant