#10847: matrix_plot can now plot subdivisions
-----------------------------------+----------------------------------------
   Reporter:  jason                |       Owner:  jason, was
       Type:  enhancement          |      Status:  needs_work
   Priority:  major                |   Milestone:  sage-4.7  
  Component:  graphics             |    Keywords:            
     Author:  Jason Grout          |    Upstream:  N/A       
   Reviewer:  Karl-Dieter Crisman  |      Merged:            
Work_issues:                       |  
-----------------------------------+----------------------------------------
Changes (by kcrisman):

  * status:  needs_review => needs_work
  * reviewer:  => Karl-Dieter Crisman


Comment:

 This is such a great patch, really nicely put together and documented,
 everything.  I really want to say positive review, except plotting sparse
 matrices now does not work.
 {{{
 sage: sparse = matrix(dict([((randint(0, 10), randint(0, 10)), 1) for i in
 xrange(100)])); matrix_plot(sparse)
 <snip>
 AttributeError: Unknown property subdivisions
 }}}
 I have no idea how to fix this - all the obvious things I tried, including
 adding to the list of options deleted for sparse plotting, or deleting
 this options after creating the lines, did not work. I must be missing
 something pretty obvious, I guess, but since this is '''one of the doctest
 examples''' it now fails the tests and obviously this needs work!

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

Reply via email to