search

threeML / hawc_hal

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

remove install root_numpy #79

Closed xiaojieww closed 2 years ago

xiaojieww commented 2 years ago

For testing uproot, remove all the installation of root_numpy. Some of the automatic tests for #78 failed, doubt may be related to there being still pip install -c conda-forge root_numpy in the automatic test script.

henrikef commented 2 years ago

@xiaojieww Not sure what you intended with this but you will either need to merge @torresramiro350 's branch into this one or this one into the branch in #78 to get the changes in the same place.

xiaojieww commented 2 years ago

@xiaojieww Not sure what you intended with this but you will either need to merge @torresramiro350 's branch into this one or this one into the branch in #78 to get the changes in the same place.

I try to make this PR work after merge #78. We didn't realize that we could merge this one into #78 . Since it related to the automatic tests. Let us check the latter way and make the changes in the same place.