Enrico Guiraud

[DF] Apply some include fixes from include-what-you-use

10

This is not all the suggested fixes, but it should be an improvement w.r.t. the current situation.

[VecOps] Do not enforce const-ness in Map function parameters

12

Before this PR, the lambda passed to VecOps::Map could not take arguments by non-const reference as the implementation was adding a const qualifier to the RVec arguments. We now instead...

[DF] Rename RDF factory functions

6

This PR changes names such as `ROOT::RDF::MakeCsvDataFrame` to the nicer/clearer convention `ROOT::RDF::FromCSV`, deprecating the previous spellings. Although what's nicer and clearer is generally subjective, I think there is strong enough...

[DF] Better name for test file

1

`NeverWrittenOut.root` is actually written out (because the exception is thrown during the event loop). Change the file name so that it's deleted at the end of the test.

[DF] Decouple type of RVariation storage from type returned by Vary expression

1

To this end, we now propagate compile-time information on how many columns are varied (one or more than one simultaneously) down from `Vary` to `RVariation`. We can take advantage of...

[DF] Move large parts of RInterface to a non-template base class

4

This is to reduce the amount of code generated for every separate RInterface instantiation. The PR should not otherwise modify any logic or functionality. I hoped this would reduce compile...

As discussed during https://indico.cern.ch/event/1176978/ :+1: - [ ] always add `EXTRA_CLING_ARGS='-O0 -fno-omit-frame-pointer' when `CLING_DEBUG=1` - [ ] suggest re-running with `CLING_DEBUG=1` in the message surrounding the ROOT-generated stack traces -...

[PyROOT] Crash when passing functor to template method

5

Given: ```python ROOT.gInterpreter.Declare(""" struct MyFunctor { bool operator()(int run) { return run > 0; }; }; """) df = ROOT.RDataFrame(10).Define("x", "42") ``` this works fine: ```python df.Define("y", ROOT.MyFunctor(), ["x"]) ```...

Reading entries on a cloned TNtuple broken if cloning happened when an unrelated file was the current file

4

The following is a reproducer: ```cpp #include #include #include int main() { TFile f0("o1.root"); TFile f1("o2.root"); TNtuple *B2L = f0.Get("B2Lc"); std::cout GetEntry(0); std::cout Clone()); f1.cd(); std::cout GetEntry(0); std::cout

Add a mechanism to plug in third-party magics

2

I would like to add a magic word to xeus-cling to implement a fairly domain-specific feature. As such, it would not make sense to propose a patch upstream, and it...