#15184: CachedFunction file location seems problematic
--------------------------+--------------------------
       Reporter:  nbruin  |        Owner:
           Type:  defect  |       Status:  needs_work
       Priority:  major   |    Milestone:  sage-5.12
      Component:  misc    |   Resolution:
       Keywords:          |    Merged in:
        Authors:          |    Reviewers:
Report Upstream:  N/A     |  Work issues:
         Branch:          |       Commit:
   Dependencies:          |     Stopgaps:
--------------------------+--------------------------
Changes (by SimonKing):

 * status:  needs_review => needs_work


Comment:

 With the patch, we get
 {{{
 sage: P.<x,y> = QQ[]
 sage: I = P*x
 sage: from sage.misc.sageinspect import _sage_getdoc_unformatted
 sage: _sage_getdoc_unformatted(I.groebner_basis)
 "File:
 /mnt/local/king/SAGE/prereleases/sage-5.12.beta5/local/lib/python2.7/site-
 packages/sage/rings/polynomial/multi_polynomial_ideal.py (starting at line
 3533)
 }}}
 and I think this is not what we want. I believe we should be as close to
 Cython's behaviour as possible, hence, it should be
 {{{
 File: sage/rings/polynomial/multi_polynomial_ideal.py
 }}}
 Currently, it is `File: /sage/rings/polynomial/multi_polynomial_ideal.py`,
 and I think according to what Cython does we should get rid of the leading
 slash.

--
Ticket URL: <http://trac.sagemath.org/ticket/15184#comment:8>
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/groups/opt_out.

Reply via email to