You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Explain what you would like to see improved and how.
This issue is meant to collect PRs working towards making the RNTupleProcessor fully composable. Here "composable" means that each type of processor can be create from other processors (e.g., an RNTupleChainProcessor can be created from several RNTupleJoinProcessors or vice-versa). Since this will involve significant changes and additions to the existing RNTupleProcessor code, it seems sensible to split this effort over multiple PRs to hopefully make it a bit easier to review and keep track of the changes.
Three main steps are foreseen:
Add an RNTupleSingleProcessor for processing single RNTuples, using the same interface as RNTuple[Chain|Join]Processor (PR: [ntuple] Add RNTupleSingleProcessor #17134).
Explain what you would like to see improved and how.
This issue is meant to collect PRs working towards making the
RNTupleProcessor
fully composable. Here "composable" means that each type of processor can be create from other processors (e.g., anRNTupleChainProcessor
can be created from severalRNTupleJoinProcessors
or vice-versa). Since this will involve significant changes and additions to the existingRNTupleProcessor
code, it seems sensible to split this effort over multiple PRs to hopefully make it a bit easier to review and keep track of the changes.Three main steps are foreseen:
RNTupleSingleProcessor
for processing single RNTuples, using the same interface asRNTuple[Chain|Join]Processor
(PR: [ntuple] AddRNTupleSingleProcessor
#17134).RNTupleChainProcessor
to useRNTuple[Single|Chain|Join]Processor
objects instead of single RNTuples directly (PR: [ntuple] MakeRNTupleChainProcessor
composable #17393).RNTupleJoinProcessor
to useRNTuple[Single|Chain|Join]Processor
objects instead of single RNTuples directly.Additional code and test cleanups will also be tracked here.
The text was updated successfully, but these errors were encountered: