Opened 3 years ago

Closed 3 years ago

#24751 closed defect (worksforme)

py3: bug in pynac when evaluating functions

Reported by: embray Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: python3 Keywords: pynac
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Fixed upstream, but not in a stable release. Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by embray)

There are several cases like:

sage: arcsinh(z)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-2-fbfc0c968329> in <module>()
----> 1 arcsinh(z)

/home/embray/src/sagemath/sage-python3/local/lib/python3.6/site-packages/sage/symbolic/function.pyx in sage.symbolic.function.BuiltinFunction.__call__ (build/cythonized/sage/symbolic/function.cpp:11719)()
    988
    989             if callable(method):
--> 990                 res = method()
    991
    992         if res is None:

/home/embray/src/sagemath/sage-python3/local/lib/python3.6/site-packages/sage/symbolic/expression.pyx in sage.symbolic.expression.Expression.arcsinh (build/cythonized/sage/symbolic/expression.cpp:49022)()
   8588         """
   8589         return new_Expression_from_GEx(self._parent,
-> 8590                 g_hold_wrapper(g_asinh, self._gobj, hold))
   8591
   8592     def arccosh(self, hold=False):

TypeError: attribute name must be string, not 'bytes'

Upstream PRs: https://github.com/pynac/pynac/pull/306 https://github.com/pynac/pynac/pull/308

Change History (7)

comment:1 Changed 3 years ago by embray

  • Report Upstream changed from Reported upstream. No feedback yet. to Fixed upstream, but not in a stable release.

comment:2 Changed 3 years ago by embray

  • Description modified (diff)

comment:3 Changed 3 years ago by chapoton

  • Milestone changed from sage-8.2 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

This seems to be no longer an issue..

comment:4 Changed 3 years ago by embray

I wouldn't say it's no longer an issue, but it should be fixed by #24838.

comment:5 Changed 3 years ago by chapoton

It seems to be fixed now, that's what I mean.. I am using a python3 branch + only a few more tickets on top of it.

comment:6 Changed 3 years ago by chapoton

  • Status changed from needs_review to positive_review

this should be closed now, please

comment:7 Changed 3 years ago by embray

  • Resolution set to worksforme
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.