threeML / hawc_hal

HAWC Accelerated Likelihood - python-only framework for HAWC data analysis
BSD 3-Clause "New" or "Revised" License
11 stars 22 forks source link

Alternative to root_numpy? #42

Closed henrikef closed 2 years ago

henrikef commented 3 years ago

root_numpy is no longer maintained. We should find an alternative package for loading root files (or switch to hdf5 entirely).

From https://github.com/scikit-hep/root_numpy:

root_numpy has not been actively maintained in several years. This is mostly due to the emergence of new alternatives which are both faster and more flexible.

  • uproot provides support for reading and writing ROOT files without the need for an installation of ROOT. See here for details.
  • ROOT now natively supports converting objects into numpy arrays using either directly using TTree or with the newer RDataFrame
github-actions[bot] commented 3 years ago

This issue has become stale. Is there an update? We will close in 14 days

github-actions[bot] commented 3 years ago

This issue has become stale. Is there an update? We will close in 14 days

henrikef commented 2 years ago

Going to re-open this because it's still an issue. hawc_hal will soon be the only part of threeML requiring root - even the fermitools have now dropped root as a dependency. @xiaojieww , I really hope you guys investigate an alternative like uproot.

torresramiro350 commented 2 years ago

I am working on removing any ROOT dependencies and implementing uproot into HAL

xiaojieww commented 2 years ago

Thanks! @torresramiro350

torresramiro350 commented 2 years ago

I think this should be closed as #78 addresses this issue.

xiaojieww commented 2 years ago

Yes. I agree this can be closed.