Ticket #2100: trac_2100-3d-vector.patch

sage/plot/plot3d/base.pyx

@@ -943 +943 @@
             opts['aspect_ratio'] = (1, 1, 1)
         if not isinstance(opts['aspect_ratio'], (str, list, tuple)):
             raise TypeError, 'aspect ratio must be a string, list, tuple, or 1'
+        # deal with any aspect_ratio instances passed from the default options to plot
+        if opts['aspect_ratio'] == 'auto':
+            opts['aspect_ratio'] = 'automatic'
         
         if opts['frame_aspect_ratio'] == 'automatic':
             if opts['aspect_ratio'] != 'automatic':
@@ -1025 +1028 @@
 
             sage: sphere((0,0,0))
 
-        EXAMPLES: We illustrate use of the aspect_ratio option::
+        EXAMPLES: We illustrate use of the ``aspect_ratio`` option::
 
             sage: x, y = var('x,y')
             sage: p = plot3d(2*sin(x*y), (x, -pi, pi), (y, -pi, pi))
@@ -1039 +1042 @@
 
             sage: p.show(aspect_ratio=[1,1,1], frame_aspect_ratio=[1,1,1/8])
 
+        This example shows indirectly that we can accept the default 
+        from :func:`~sage.plot.plot.plot` of ``aspect_ratio='auto'``, if desired::
+
+            sage: plot(vector([1,2,3]))
+
         We use the 'canvas3d' backend from inside the notebook to get a view of
         the plot rendered inline using HTML canvas::
 

sage/plot/plot3d/plot_field3d.py

@@ -57 +57 @@
         sage: plot_vector_field3d((x*cos(z),-y*cos(z),sin(z)), (x,0,pi), (y,0,pi), (z,0,pi),plot_points=4)
         sage: plot_vector_field3d((x*cos(z),-y*cos(z),sin(z)), (x,0,pi), (y,0,pi), (z,0,pi),plot_points=[3,5,7])
         sage: plot_vector_field3d((x*cos(z),-y*cos(z),sin(z)), (x,0,pi), (y,0,pi), (z,0,pi),center_arrows=True)
+
+    TESTS:
+
+    This tests that Trac # 2100 is fixed in a way compatible with this command::
+
+        sage: plot_vector_field3d((x*cos(z),-y*cos(z),sin(z)), (x,0,pi), (y,0,pi), (z,0,pi),center_arrows=True,aspect_ratio=(1,2,1))
     """
     (ff,gg,hh), ranges = setup_for_eval_on_grid(functions, [xrange, yrange, zrange], plot_points)
     xpoints, ypoints, zpoints = [srange(*r, include_endpoint=True) for r in ranges]