-
Notifications
You must be signed in to change notification settings - Fork 54
Update with python 3.7 compatible cython #368
Conversation
|
Is there an update on the status of this pull request? With some services removing support for python 3.5 it would be very useful to have a 3.7 compatible version of root_numpy. |
You should definitely note that the build failed in python 2.7 and ROOT6. I might also ask what you're using root_numpy for as things are most likely slowly moving to support uproot instead. |
|
For what it's worth, I believe the python 2.7 with root 6 tests were failing before these changes |
|
Right, so I don't mind merging this in, but can you test on python 2.7? Virtualenvironments on your machine should make this easy. |
|
I have just been using it to read into a root tree and convert it into an array, from the looks of uproot that should do the same, I will give that a test. Thank you. |
|
Beyond opening files (which In any case, since PyROOT depends on the python version, just using a virtualenv isn't sufficient. However, I now have a python 2.7 + ROOT 6.10.08 docker container (that I didn't have when I created this PR). I just ran the tests (log below) and they looks fine to me: |
|
Thanks! |
See discussion in #365. I followed the suggestions there to regenerate these files with cython and it seemed to work. As noted there, I haven't been able to test on other python / ROOT versions other than python 3.7 with ROOT 6.14.