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
@ariostas, it sounds like the TTree (HasBranches) keys method was copied without enough modification. HasBranches.keys has a filter_branches to select branches, but maybe the equivalent is called "fields" for RNTuple? Or it doesn't exist (filter_branches takes a function on TBranch objects, which maybe aren't objects for fields?)?
Yeah, the filter_branches keyword argument doesn't make sense for RNTuples, but it would make sense to have a filter_fields now that there are field objects.
The current implementation of keys is not fully-featured yet. Here's how it looks.
# TODO: some arguments might be missing when compared with TTree. Solve when blocker is present in dask/coffea.
):
The filter_typename, recursive, and full_paths don't do anything yet. I'll have to check how recursive, and full_paths work for TTrees because I'm not sure how they translate to RNTuples.
Trying https://github.com/iris-hep/calver-coffea-agc-demo with one of RNTuple file (v1) - to reproduce it, change in cell 5
we see next error:
cc: @alexander-held
The text was updated successfully, but these errors were encountered: