root_numpy icon indicating copy to clipboard operation
root_numpy copied to clipboard

Check fails with Python 3.7.2

Open kgizdov opened this issue 6 years ago • 5 comments

On ArchLinux with GCC 8.2.1, Python 3.7.2, when running:

nosetests -s -v root_numpy

I get the following errors:

======================================================================
ERROR: root_numpy.tests.test_evaluate.test_evaluate_func
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/gizdov/builds/python-root_numpy/src/root_numpy-4.8.0/root_numpy/tests/test_evaluate.py", line 43, in test_evaluate_func
    assert_raises(ValueError, rnp.evaluate, "f", arr_1d)
  File "/usr/lib/python3.7/unittest/case.py", line 743, in assertRaises
    return context.handle('assertRaises', args, kwargs)
  File "/usr/lib/python3.7/unittest/case.py", line 178, in handle
    callable_obj(*args, **kwargs)
  File "/home/gizdov/builds/python-root_numpy/src/root_numpy-4.8.0/root_numpy/_evaluate.py", line 105, in evaluate
    obj = ROOT.TFormula(uuid.uuid4().hex, obj)
TypeError: none of the 4 overloaded methods succeeded. Full details:
  TFormula::TFormula(const char* name, const char* formula = "", bool addToGlobList = true, bool vectorize = false) =>
    problem in C++; program state has been reset
  TFormula::TFormula(const char* name, const char* formula, int ndim, int npar, bool addToGlobList = true) =>
    takes at least 4 arguments (2 given)
  TFormula::TFormula() =>
    takes at most 0 arguments (2 given)
  TFormula::TFormula(const TFormula& formula) =>
    takes at most 1 arguments (2 given)

----------------------------------------------------------------------
Ran 227 tests in 2.944s

FAILED (errors=1)

kgizdov avatar Mar 27 '19 15:03 kgizdov

Which ROOT are you reporting this for?

kratsg avatar Mar 27 '19 15:03 kratsg

ROOT is at v6.16.00

kgizdov avatar Mar 27 '19 16:03 kgizdov

Seems like they changed the TFormula interface here. Can you downgrade ROOT? if you seriously want to use this (support-dropped) package, you need to use 6.12-ish as the latest. Otherwise, if you want the newer ROOT, switch to uproot.

kratsg avatar Mar 27 '19 17:03 kratsg

@kratsg if this package is deprecated, it should be reflected in the docs or something. I was unaware it's no longer in development.

kgizdov avatar Mar 27 '19 17:03 kgizdov

There's no active support -- or official maintainer. It's mostly held up by people volunteering their time now.

kratsg avatar Mar 27 '19 17:03 kratsg

The package is deprecated. Closing as won't do.

eduardo-rodrigues avatar Jan 05 '23 15:01 eduardo-rodrigues