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

Coercion problem with QQbar #12665

Closed
loefflerd mannequin opened this issue Mar 13, 2012 · 6 comments
Closed

Coercion problem with QQbar #12665

loefflerd mannequin opened this issue Mar 13, 2012 · 6 comments
Assignees
@aghitza
Labels
c: algebra p: major / 3 t: bug
Milestone
sage-5.0

Comments

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Mar 13, 2012

I spotted this while working on #12662:

QQbar(sqrt(2)) + QQbar(zeta(7))

will return a massive traceback concluding in

RuntimeError: maximum recursion depth exceeded while calling a Python object
sage:

Component: algebra

Author: Mike Hansen

Reviewer: David Loeffler

Merged: sage-5.0.beta11

Issue created by migration from https://trac.sagemath.org/ticket/12665
@loefflerd loefflerd mannequin added this to the sage-5.0 milestone Mar 13, 2012
@loefflerd loefflerd mannequin added c: algebra labels Mar 13, 2012
@loefflerd loefflerd mannequin assigned aghitza Mar 13, 2012
@mwhansen
Copy link
Contributor

Author: Mike Hansen

@mwhansen
Copy link
Contributor

comment:1

Attachment: trac_12665.patch.gz

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Mar 25, 2012

Reviewer: David Loeffler

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Mar 25, 2012

comment:2

Looks good and fixes the problem. Positive review.

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Mar 25, 2012

comment:3

For the record, while reviewing this patch I noticed that QQbar.zeta(6) also fails, in a completely different way! See followup ticket #12746.

@jdemeyer
Copy link
Contributor

Merged: sage-5.0.beta11

This was referenced Mar 28, 2012
Closed
Open
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aghitza aghitza

Labels
c: algebra p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.0
Development

No branches or pull requests
3 participants
@mwhansen @aghitza @jdemeyer