#17578: Use Parent/Element for Manin symbols
-------------------------------------+-------------------------------------
       Reporter:  pbruin             |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.5
      Component:  modular forms      |   Resolution:
       Keywords:  Manin symbol       |    Merged in:
        Authors:  Peter Bruin        |    Reviewers:  Travis Scrimshaw
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/tscrim/manin_symbol_parent-17578 |  57a425901538723e212815ab1e84908ff992cf40
   Dependencies:  #17579             |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by tscrim):

 Replying to [comment:13 was]:
 > If I'm reading your test, that's not just "a slowdown...we might just
 have to deal with", it's an '''epic slowdown''', by a full factor of 10.
 Am I reading that right?  If so, then this is definitely seriously
 worrisome.

 The first line is the actual creation, the second is when the caching
 comes into effect. However the slowdown is by ~1.4x due to the overhead.
 Thus if we want to fix the probthese without the overhead from the
 category, we need an entirely different approach. Do you think you could
 run a timing in a more realistic computation or post such a computation?

--
Ticket URL: <http://trac.sagemath.org/ticket/17578#comment:14>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to