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

Lazy fields do not support abs() #11870

Closed
jdemeyer opened this issue Sep 29, 2011 · 6 comments
Closed

Lazy fields do not support abs() #11870

jdemeyer opened this issue Sep 29, 2011 · 6 comments

Comments

@jdemeyer
Copy link

sage: RLF(1).abs()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)

/usr/local/src/sage-4.7.2.alpha2/<ipython console> in <module>()

/usr/local/src/sage-4.7.2.alpha2/local/lib/python2.6/site-packages/sage/structure/element.so in sage.structure.element.RingElement.abs (sage/structure/element.c:13785)()

/usr/local/src/sage-4.7.2.alpha2/local/lib/python2.6/site-packages/sage/rings/real_lazy.so in sage.rings.real_lazy.LazyFieldElement.__getattribute__ (sage/rings/real_lazy.c:7550)()

/usr/local/src/sage-4.7.2.alpha2/local/lib/python2.6/site-packages/sage/structure/element.so in sage.structure.element.Element.__getattr__ (sage/structure/element.c:2794)()

/usr/local/src/sage-4.7.2.alpha2/local/lib/python2.6/site-packages/sage/structure/parent.so in sage.structure.parent.getattr_from_other_class (sage/structure/parent.c:2940)()

/usr/local/src/sage-4.7.2.alpha2/local/lib/python2.6/site-packages/sage/structure/parent.so in sage.structure.parent.raise_attribute_error (sage/structure/parent.c:2709)()

AttributeError: 'sage.rings.real_lazy.LazyWrapper' object has no attribute '__abs__'

Same for CLF.

Component: calculus

Author: Jeroen Demeyer

Reviewer: Burcin Erocal

Merged: sage-4.7.2.alpha4

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

@jdemeyer
Copy link
Author

Attachment: 11870.patch.gz

@burcin
Copy link

burcin commented Oct 7, 2011

comment:2

Looks good to me.

@burcin
Copy link

burcin commented Oct 7, 2011

Reviewer: Burcin Erocal

@burcin
Copy link

burcin commented Oct 7, 2011

Author: Jeroen Demeyer

@jdemeyer
Copy link
Author

jdemeyer commented Oct 7, 2011

comment:3

Thanks!

@jdemeyer
Copy link
Author

jdemeyer commented Oct 7, 2011

Merged: sage-4.7.2.alpha4

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