#9740: matrix plot is upside down and should wrap more matplotlib options
---------------------------+------------------------------------------------
   Reporter:  jason        |       Owner:  jason, was  
       Type:  enhancement  |      Status:  needs_review
   Priority:  major        |   Milestone:  sage-4.6    
  Component:  graphics     |    Keywords:              
     Author:  Jason Grout  |    Upstream:  N/A         
   Reviewer:               |      Merged:              
Work_issues:               |  
---------------------------+------------------------------------------------

Comment(by kcrisman):

 Okay, in general this is great!  I love the live documentation for
 testing, by the way - don't know why I never thought of that before.

 Two things, which perhaps should still be addressed - what do you think?

 First, the error message with the `matrix_plot(A,marker=',')` is not very
 helpful when you choose a different marker (for instance, '<' works (and
 is cool with small matrices) but ';' doesn't).  I realize this isn't part
 of the ticket per se, so maybe this should be a followup.

 Second, there are two instances of
 {{{
 Extra options will get passed on to show(), as long as they are valid:
 }}}
 in the documentation for `MatrixPlot`.  Again, maybe this is better
 addressed on a followup ticket.

 I like that the error message for when `vmin` and `vmax` clash is actually
 helpful, even though it's an mpl error.

 I also just noticed that the doc for `matrix_plot` doesn't mention that
 the default for `norm` is `None`, though it is.  This can probably be
 fixed very easily.

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