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

Implement conjugate() for RealDoubleElement #7834

Closed
sagetrac-dagss mannequin opened this issue Jan 3, 2010 · 4 comments
Closed

Implement conjugate() for RealDoubleElement #7834

sagetrac-dagss mannequin opened this issue Jan 3, 2010 · 4 comments
Assignees
@aghitza
Labels
c: basic arithmetic p: major / 3 t: bug
Milestone
sage-4.3.1

Comments

@sagetrac-dagss
Copy link
Mannequin

sagetrac-dagss mannequin commented Jan 3, 2010

This appears inconsistent, and is an actual problem for me:

sage: ZZ(4).conjugate()
4
sage: RR(4).conjugate()
4.00000000000000
sage: RDF(4).conjugate()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)

/uio/arkimedes/s07/dagss/.sage/temp/corcaroli.uio.no/12687/_uio_arkimedes_s07_dagss__sage_init_sage_0.py in <module>()

AttributeError: 'sage.rings.real_double.RealDoubleElement' object has no attribute 'conjugate'

Component: basic arithmetic

Author: Dag Sverre Seljebotn

Reviewer: Alex Ghitza

Merged: sage-4.3.1.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/7834
@sagetrac-dagss sagetrac-dagss mannequin added this to the sage-4.3.1 milestone Jan 3, 2010
@sagetrac-dagss sagetrac-dagss mannequin assigned aghitza Jan 3, 2010
@sagetrac-dagss
Copy link
Mannequin Author

sagetrac-dagss mannequin commented Jan 3, 2010

comment:1

Attachment: RDF_conjugate.patch.gz

@sagetrac-dagss sagetrac-dagss mannequin added the s: needs review label Jan 3, 2010
@aghitza
Copy link

aghitza commented Jan 3, 2010

Reviewer: Alex Ghitza

@aghitza
Copy link

aghitza commented Jan 3, 2010

comment:2

Looks good to me.

@mwhansen
Copy link
Contributor

mwhansen commented Jan 4, 2010

Merged: sage-4.3.1.alpha0

@mwhansen mwhansen closed this as completed Jan 4, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aghitza aghitza

Labels
c: basic arithmetic p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.3.1
Development

No branches or pull requests
2 participants
@mwhansen @aghitza