#11342: Make getattr faster on parents and elements
------------------------------+---------------------------------------------
Reporter: SimonKing | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: performance | Keywords: getattr parent element
Work_issues: | Upstream: N/A
Reviewer: Jeroen Demeyer | Author: Simon King
Merged: | Dependencies: #9944
------------------------------+---------------------------------------------
Comment(by SimonKing):
Replying to [comment:45 vbraun]:
> Ok I looked at `sage/rings/pari_ring.py` and it makes my eyes bleed...
The elements don't call `RingElement.__init__` so their parent is not set.
Also, should use `UniqueRepresentation`. But is anybody using that module?
I guess, implicitly we are using it all: As we have seen, some pari
objects are generated when sage starts. And you are right, it is evident
that sage/rings/pari_ring.py has been written long time ago...
Do you think we should attempt to bring it up to date ''here''? Or do it
on a different ticket and just use the work-around provided by
[attachment:trac_11342_fix_pari_initialization.patch]?
If we fix it here: Did you already start with it? Otherwise, I'd do it
myself.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11342#comment:46>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
