FunTuple - API changes and simplification of Python helper modules
Hi @amathad, I'm making a simplification here to avoid a packing-unpacking in what is anyway an internal function.
While at it I suggest the renaming tree_name -> tuple_name as I really hope we will move from TTree to RNTuple at some point. I realise this breaks the API and requires updates in DaVinci, but I would rather do it now than 2 months down the line. What do you think?
More details on the API changes in the related DaVinci MR DaVinci!652 (merged).
Merge request reports
Activity
assigned to @erodrigu
- Resolved by Patrick Koppenburg
As discussed today, this sounds good to me. Indeed all examples in DV aswell as a change in FunTuple here is needed.
Edited by Abhijit Mathad
added 1 commit
- ffdc27b8 - Replace tree_name -> tuple_name throughought
mentioned in merge request DaVinci!637 (merged)
- Resolved by Eduardo Rodrigues
- Resolved by Eduardo Rodrigues
/ci-test --merge
- Resolved by Eduardo Rodrigues
(or does this require some DaVinci MR?)
added ci-test-triggered label
- [2022-02-04 11:26] Validation started with lhcb-master-mr#3623
- [2022-02-10 08:18] Validation started with lhcb-master-mr#3678
- [2022-02-10 11:39] Validation started with lhcb-master-mr#3680
Edited by Software for LHCbadded 1 commit
- f2d6350e - Rename branch -> field having in mind RNTuple in the future!