#12630: Add representations of quivers and quiver algebras to sage
------------------------------------------------------------------------------+
       Reporter:  JStarx                                                      | 
        Owner:  AlexGhitza                                                      
      
           Type:  enhancement                                                 | 
       Status:  needs_work                                                      
      
       Priority:  major                                                       | 
    Milestone:  sage-5.11                                                       
      
      Component:  algebra                                                     | 
   Resolution:                                                                  
      
       Keywords:  algebra, quiver, module, days49                             | 
  Work issues:  Use existing infrastructure for Representation, Homspace and 
Morphism.
Report Upstream:  N/A                                                         | 
    Reviewers:                                                                  
      
        Authors:  Jim Stark, Simon King, Mathieu Guay-Paquet, Aladin Virmaux  | 
    Merged in:                                                                  
      
   Dependencies:  #12412, #12413                                              | 
     Stopgaps:                                                                  
      
------------------------------------------------------------------------------+

Comment (by SimonKing):

 Solution of the problem: In get_matrix, there is
 {{{
 list(self._vector[startdim:startdim + rows*cols])
 }}}
 If `__getitem__` of `self._vector` is asked to slice (which is the case
 here), then it first creates the list and then creates a vector out of a
 slice of that list (also creating a parent, I guess). And from the
 resulting vector, we take the list.

 It should instead be
 {{{
 self._vector.list()[startdim:startdim + rows*cols]
 }}}
 Then, `list()` is called only once, and no temporary slice vector is
 created.

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