#20026: Reverse the scale of colors in Matrix.plot
-------------------------+-------------------------------------------------
Reporter: | Owner:
ncohen | Status: needs_info
Type: | Milestone: sage-7.1
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Jori Mäntysalo
linear algebra | Work issues:
Keywords: | Commit:
Authors: | 791fe5552026c199a7ce575bc594a8e705972549
Nathann Cohen | Stopgaps:
Report Upstream: N/A |
Branch: |
public/20026 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by ncohen):
> I don't feel like it's really appropriate to change this without some
extra discussion somewhere. It's clearly not a bug. Remember, long-time
"customers" may be used to this behavior (I can't even find a ticket for
when this was added!), so a change within five hours seems a little on the
quick side. E.g., what is matplotlib's actual default for `imshow()` in
this situation?
The default behaviour seems to be what you get by setting 'cmap' to None:
{{{
sage: matrix.random(IntegerModRing(5),5,5).plot(cmap=None)
}}}
It also matches the examples given there:
http://matplotlib.org/examples/pylab_examples/image_interp.html
> And the branch doesn't even mention any changes in the documentation
alerting people that this is now different!
If you see a smart way to do it, you are welcome to add a commit. I'm 100%
sure that none of those who call this method will read the docstring
*before* they notice the change. And if they do read the doc afterwards,
all they can learn there is "Yeah, there has been a change".
> Compare to #18463 which (eventually) fixed an actual bug.
This ticket seems unrelated to the color scale.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/20026#comment:7>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
