#16296: Speed improvements for categories with axioms
-------------------------------------+-------------------------------------
       Reporter:  SimonKing          |        Owner:
           Type:  enhancement        |       Status:  new
       Priority:  major              |    Milestone:  sage-6.3
      Component:  categories         |   Resolution:
       Keywords:  cython             |    Merged in:
  performance categories             |    Reviewers:
        Authors:                     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  9a420ad15fc941dfd3b3a12d005b4f814407a008
  u/SimonKing/ticket/16296           |     Stopgaps:
   Dependencies:  #10963, #15801,    |
  #16309                             |
-------------------------------------+-------------------------------------

Comment (by SimonKing):

 Now I get what is going wrong.

 Since `Rings.HomCategory` has no docstring, `_sage_getdoc_unformatted`
 returns the docstring of `Rings.HomCategory.__init__`. This used to by a
 Python method, hence had no embedded information on the source code. But
 with this branch, `__init__` is a Cython information and thus has
 embedding information, that unfortunately lets `sage_getsourcelines` look
 up the wrong file.

 But in the first place: Why do we need to look up
 `_sage_getdoc_unformatted` when calling `sage_getsourcelines`? After all,
 `Rings.HomCategory` is a Python class in a Python file! Shouldn't it
 provide all relevant information?

--
Ticket URL: <http://trac.sagemath.org/ticket/16296#comment:22>
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