Add a 'sink' parameter to the LazyTree() functions to support other table/dataframe types

[grasph]

closes #280

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Files	Coverage Δ
src/iteration.jl	85.54% <100.00%> (+0.52%)	⬆️

📢 Thoughts on this report? Let us know!.

[Moelf]

according to documentation

which wraps any Tables.AbstractColumns-compatible object that has already had copies of its columns made, and are thus safe for the columns-consumer to assume ownership of (this is because DataFrames.jl, by default makes copies of all columns upon construction). In both cases, individual columns are collected in Vector{AbstractVector}s by calling Tables.getcolumn(x, nm) for each column name.

are you sure our getcolumn doesn't allocated?

[Moelf]

Looks good to me

this PR

julia> @time LazyTree("./test/samples/NanoAODv5_sample.root", "Events", r"Muon.*");
  0.041865 seconds (394.89 k allocations: 39.741 MiB)

julia> @time LazyTree("./test/samples/NanoAODv5_sample.root", "Events", r"Muon.*"; sink=DataFrame);
  0.041879 seconds (394.97 k allocations: 39.748 MiB)

main

julia> @time LazyTree("./test/samples/NanoAODv5_sample.root", "Events", r"Muon.*");
  0.045695 seconds (394.95 k allocations: 39.757 MiB)

julia> @time DataFrame(LazyTree("./test/samples/NanoAODv5_sample.root", "Events", r"Muon.*"); copycols=false);
  0.045971 seconds (395.03 k allocations: 39.766 MiB)

julia> @time DataFrame(LazyTree("./test/samples/NanoAODv5_sample.root", "Events", r"Muon.*"); copycols=true);
  0.066453 seconds (398.93 k allocations: 43.543 MiB)

[Moelf]

@grasph I hope you're okay with the tests and cosmetics changes I made, I think we can safely merge this and make a patch release.

Minor version Y (x.Y.z | x > 0) MUST be incremented if new, backwards compatible functionality is introduced to the public API.

we are pre 1.0 (x == 0) so I say we don't bump the 10 in v0.10.z