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.

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

atan2(-pi,0) throws "divide by zero" #8565

Closed
burcin opened this issue Mar 20, 2010 · 7 comments
Closed

atan2(-pi,0) throws "divide by zero" #8565

burcin opened this issue Mar 20, 2010 · 7 comments

Comments

@burcin
Copy link

burcin commented Mar 20, 2010

From sage-devel:

the summary is:
-------------------
atan2(3,0)   --> 1/2*pi
atan2(-3,0)  --> -1/2*pi
atan2(pi,0)  --> 1/2*pi
atan2(-pi,0) -->  RuntimeError: power::eval(): division by zero
--------------------

sage-devel thread is here:

http://groups.google.com/group/sage-devel/t/317e6bfe11fabb4

also reported on sage-support:

http://groups.google.com/group/sage-support/t/02f3446e68381346

Component: symbolics

Keywords: pynac

Author: Burcin Erocal, Richard Kreckel, Jens Vollinga

Reviewer: Robert Mařík

Merged: sage-4.4.1.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/8565

@burcin burcin self-assigned this Mar 20, 2010
@burcin
Copy link
Author

burcin commented Apr 2, 2010

add doctests

@burcin
Copy link
Author

burcin commented Apr 2, 2010

Author: Burcin Erocal, Richard Kreckel, Jens Vollinga

@burcin
Copy link
Author

burcin commented Apr 2, 2010

comment:1

Attachment: trac_8565-neg_pi.patch.gz

The pynac package at #8644 includes the patches that were merged in GiNaC to fix this problem. attachment: trac_8565-neg_pi.patch adds doctests for the fix.

This ticket now depends on #8644.

@burcin burcin added this to the sage-4.4 milestone Apr 2, 2010
@robert-marik
Copy link
Mannequin

robert-marik mannequin commented Apr 9, 2010

comment:2

Installs fine, all tests passed, works ad advertised. Positive review and thanks for fixing.

@robert-marik
Copy link
Mannequin

robert-marik mannequin commented Apr 9, 2010

Reviewer: Robert Mařík

@williamstein
Copy link
Contributor

Merged: 4.4.1.alpha2

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 3, 2010

Changed merged from 4.4.1.alpha2 to sage-4.4.1.alpha2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants