make Graphics.plot refuses argument

[vdelecroix]

Currently Graphics.plot ignore all input arguments which make the following very confusing

sage: S = circle((0,0), 1)
sage: T = S.plot(aspect_ratio=2)
sage: T.aspect_ratio()
1.0

We simply disallow arguments in order to have

sage: S.plot(aspect_ratio=1)
Traceback (most recent call last):
...
TypeError: plot() got an unexpected keyword argument 'aspect_ratio'

[vdelecroix]

New commits:

​e374128

Trac 19539: remove args and kwds in Graphics.plot

[ncohen]

Just adding a comment to say that I admire your courage.

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​03dbb2e

Trac 19539: fix doctests

[vdelecroix]

Replying to ncohen:

Just adding a comment to say that I admire your courage.

What about a courageous review?

[ncohen]

What about a courageous review?

Let's try:

-    if hasattr(funcs, 'plot'):
+    if isinstance(funcs, Graphics):
+        G = funcs
+    elif hasattr(funcs, 'plot'):

Why would you ignore (without warning) the contents of *args and **kwds when 'funcs' is a Graphics object? It seems as bad as what you try to fix in this branch.

Nathann

[vdelecroix]

More or less. Graphics is precisely the class that implemented

def plot(self, *args, **kwds):
    return self

And nobody inherits from it.

I did it because some code is calling plot(x)... you are right that it would be better to fix it instead.

What do you think that the following should do

sage: C = circle((0,0), 1)
sage: plot(C, aspect_ratio=2)

It used to return C unchanged and it still does. What my code is modifying is

sage: C.plot(aspect_ratio=2)
... -> error

[ncohen]

Hello,

What do you think that the following should do

sage: C = circle((0,0), 1)
sage: plot(C, aspect_ratio=2)

It used to return C unchanged and it still does. What my code is modifying is

If it ignores the 'aspect_ratio=2' argument then it should raise an exception instead, exactly as you do it in this branch for C.plot(whatever=3)?

Nathann

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​41dcc53	Trac 19539: revert the change to the function plot
​82f3805	Trac 19539: fix ode_solver.plot_solutions

[vdelecroix]

done!

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​a322663

Trac 19539: fix more doctests

[ncohen]

Looks good. Thank you for this branch,

Nathann