#5567: [with patch, positive review] bug in region_plot
----------------------+-----------------------------------------------------
Reporter: whuss | Owner: whuss
Type: defect | Status: closed
Priority: major | Milestone: sage-3.4.2
Component: graphics | Resolution: fixed
Keywords: |
----------------------+-----------------------------------------------------
Comment(by jason):
Replying to [comment:7 wcauchois]:
> Replying to [comment:6 jason]:
> > Shouldn't the code in the doctest be deprecated in light of
http://trac.sagemath.org/sage_trac/ticket/5413 ?
> >
> > In fact, the equify function seems like it is in direct conflict with
the deprecation in http://trac.sagemath.org/sage_trac/ticket/5413, is it
not?
>
> Playing around in the REPL, I don't see a way to define an inequality
with explicit variables -- since its not really a function. `f(x) = x^2 <
2` throws an exception, for example. For region_plot, the variables can be
made explicit by specifying them in the plot ranges, in which case they
are passed on to equify via an argument. So the ambiguity which was the
motivation for #5413 can be avoided. I think that this is a subtly
different situation, and that `equify` is fine.
The point is that equify lets variables default to None, and in that case,
automatically chooses the order of variables in the call signature.
That's what the deprecation is about---making sure that the user always
specified the order of evaluation, and not automatically choosing an
order.
And while the ambiguity can be avoided in the function call, the doctest
in the patch still uses deprecated syntax (which should throw a
deprecation warning). The doctest should still be changed to have the
variables specified.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5567#comment:8>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
