#16570: Animate example looks broken
-------------------------------------+-------------------------------------
       Reporter:  gagern             |        Owner:
           Type:  defect             |       Status:  needs_work
       Priority:  minor              |    Milestone:  sage-6.4
      Component:  graphics           |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Martin von Gagern  |    Reviewers:  Jakob Kroeker
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/gagern/ticket/16570              |  050c3b8afb736b20e46feb85f9273fba35336688
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jakobkroeker):

 * status:  needs_review => needs_work
 * reviewer:   => Jakob Kroeker


Comment:

 good, after I updated the ffmpeg 0.6.5 manually on my machine (Scientific
 Linux 6.5), the doctests passed!

 Otherwise ffmpeg 0.6.5. complains about the 'long' parameter argument.
 This might be a different new(?) issue, since several universities have SL
 installations. (SL packages are dated)

 All fixed examples worked as expected. However, there are two other broken
 animate examples;
 would you mind to fix them in this ticket? (see below)

 What also should be changed (maybe also in this ticket, since it is a
 pretty small change),
 is storing the name of the temporary directory in variable named 'dir'.
 'dir' is a keyword/a method
 and should not be overwritten

 {{{
 ############# # problem with the varying axis range #############
 #################### line 417

 a = animate([plot(x^2 + n) for n in range(4)])
 a.show()

 #################### line 449
 E = EllipticCurve('37a')
 v = [E.change_ring(GF(p)).plot(pointsize=30) for p in [97, 101, 103, 107]]
 a = animate(v, xmin=0, ymin=0)
 a
 a.show() # optional -- ImageMagick
 }}}
 There is also one example with unreadable axis labels:
 {{{
 ##############   problem with the axis labels - unreadable #############
 ############# line 294
 a = animate([circle((i,0),1) for i in srange(0,2,0.4)],
                 xmin=0, ymin=-1, xmax=3, ymax=1, figsize=[2,1])
 a.show()        # optional -- ImageMagick
 b = animate([circle((0,i),1,hue=0) for i in srange(0,2,0.4)],
                 xmin=0, ymin=-1, xmax=1, ymax=3, figsize=[1,2])
 b.show()        # optional -- ImageMagick
 }}}

 and two examples with unfortunate axis range, but that is probably
 discussable:
 {{{
 #############   image section not optimal: #############
 #############  line 326
 a = animate([circle((i,0),1,thickness=20*i) for i in srange(0,2,0.4)],
                 xmin=0, ymin=-1, xmax=3, ymax=1, figsize=[2,1],
 axes=False)
 a.show()             # optional -- ImageMagick
 b = animate([circle((0,i),1,hue=0,thickness=20*i) for i in
 srange(0,2,0.4)],
                 xmin=0, ymin=-1, xmax=1, ymax=3, figsize=[1,2],
 axes=False)
 b.show()             # optional -- ImageMagick
 p = a*b              # indirect doctest
 len(a), len(b)
 len(p)
 (a*b).show()         # optional -- ImageMagick

 #################### line 352
 a = animate([circle((i,0),1,thickness=20*i) for i in srange(0,2,0.4)],
              xmin=0, ymin=-1, xmax=3, ymax=1, figsize=[2,1], axes=False)
 a.show()
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/16570#comment:17>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to