Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

symbolic floor,ceil,factorial need _evalf_ too #18091

Closed
rwst opened this issue Mar 31, 2015 · 6 comments
Closed

symbolic floor,ceil,factorial need _evalf_ too #18091

rwst opened this issue Mar 31, 2015 · 6 comments
Labels
c: symbolics p: major / 3 t: bug
Milestone
sage-6.6

Comments

@rwst
Copy link
Contributor

rwst commented Mar 31, 2015

With

sage: f(x) = floor(x)
sage: g(x) = ceil(x)
sage: h(x) = factorial(x)

any of the following

sage: f(pi)._numerical_approx()
sage: g(pi)._numerical_approx()
sage: h(5)._numerical_approx()

will result in

src/sage/symbolic/expression.pyx in sage.symbolic.expression.Expression._numerical_approx (build/cythonized/sage/symbolic/expression.cpp:27487)()
   4888         # in pynac
   4889         if is_a_numeric(x._gobj):
-> 4890             res = py_object_from_numeric(x._gobj)
   4891         elif  is_a_constant(x._gobj):
   4892             res = x.pyobject()

TypeError: cannot evaluate symbolic expression numerically

Component: symbolics

Author: Ralf Stephan

Branch/Commit: 0db18c4

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/18091
@rwst rwst added this to the sage-6.6 milestone Mar 31, 2015
@rwst
Copy link
Contributor Author

rwst commented Mar 31, 2015

Branch: u/rws/symbolic_floor_ceil_factorial_need__evalf__too

@rwst
Copy link
Contributor Author

rwst commented Mar 31, 2015

New commits:

0db18c418091: symbolic floor,ceil,factorial need `_evalf_` too

@rwst
Copy link
Contributor Author

rwst commented Mar 31, 2015

Author: Ralf Stephan

@rwst
Copy link
Contributor Author

rwst commented Mar 31, 2015

Commit: 0db18c4

@vbraun
Copy link
Member

vbraun commented Mar 31, 2015

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Mar 31, 2015

Changed branch from u/rws/symbolic_floor_ceil_factorial_need__evalf__too to 0db18c4

@vbraun vbraun closed this as completed in 6c80105 Mar 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: symbolics p: major / 3 t: bug
Projects
None yet
Milestone
sage-6.6
Development

No branches or pull requests
2 participants
@vbraun @rwst