search

vepadulano / PyRDF

Python Library for doing ROOT RDataFrame analysis
https://pyrdf.readthedocs.io/en/latest/
9 stars 7 forks source link

Remove Local backend from PyRDF #109

Closed vepadulano closed 3 years ago

vepadulano commented 3 years ago

The Local backend is just a wrapper around ROOT.RDataFrame. It doesn't add any functionality (actually it only supports a subset of the original features) and it more a convenience in PyRDF than a necessity. Even more, in light of the integration of this package into ROOT, a ROOT user wanting to run an RDataFrame analysis on their laptop should just write code with ROOT.RDataFrame .

This commit removes the Local backend from PyRDF, along with a few modifications where needed. Notably, the Dist backend previously fell back to running with Local when the input tree had zero entries, now raises RuntimeError. Also the PyRDF.use function now only supports selecting distributed backends (for the moment limited to Spark).

Tests have been removed or changed accordingly to the modifications above.

vepadulano commented 3 years ago

Merging this PR. Changes to documentation for selecting backend will be addressed in #110 . Changes to the naming scheme will be addressed later