On May 19, 1:15 pm, P Purkayastha <ppu...@gmail.com> wrote: > On Saturday, May 19, 2012 7:35:08 PM UTC+8, Andrzej Giniewicz wrote: > > > Today one of friends I introduced to Sage noticed that > > cylindrical_plot3d does not check for it's argument like plot3d does. > > It was a result of "common beginner mistake", i.e. forgetting to > > specify "return" in function. While > > > var("x,y") > > def f(x,y): > > 1 > > plot3d(f,(x,-1,1),(y,-1,1)) > > > says "TypeError: a float is required", doing > > > cylindrical_plot3d(f,(x,-1,1),(y,-1,1)) > > > results in "RuntimeError: Segmentation fault". Wanted to ask if anyone > > can confirm this before I open ticket on trac, as like with any > > segfault I'm not sure if this is related to my build of Sage or it is > > general problem. This was checked in (almost) Sage 5.0 on x86_64 > > (although I have 3 packages patched from 12760, 12761 and 12762 and > > external GCC/ATLAS - due to decisions of package manager on my > > distribution). > > > Cheers, > > Andrzej. > > Although it looks like a bug, in both cases the error stems from > sage/plot/plot3d/parametric_surface.c (actually via the cython file > parametric_surface.pyx). So, there are no checks being performed at all.
So, are you saying that this is a check which should be performed, at least to provide a more helpful error message? We do have a bug days coming up - not that there isn't already enough work to do there :) but the more the merrier. -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org