On 02/23/2014 03:32 PM, Jeroen Demeyer wrote:
On 2014-02-22 16:58, P Purkayastha wrote:
Add another ticket to that list. SPARC also blocks an improvement in
#12798.
I totally disagree with that. That ticket has *nothing* at all to do
with SPARC. There's an obvious right fix (not rendering plots with
NaNs). If you write code which depends on the fact that a 3D renderer
will work correctly if you feed it NaNs, that's dangerous in any case.
Well, the discussion in the ticket goes into more details. But here is
the summary
1. *we* are not generating NaNs in Sage code. Those are happening from
matplotlib:
https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/delaunay/interpolate.py#L81
2. I am definitely not interested in rewriting / reimplementing working
matplotlib code.
3. The bad behavior happens *only* on SPARC and *only* during doctest.
4. The current output is *wrong*. And it has been so ever since the
function was introduced six years ago.
Personally, incorrect output is something that I am definitely not
supportive of. It is worse than not having a functionality to begin with.
--
You received this message because you are subscribed to the Google Groups
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/groups/opt_out.
