#12827: Expand Animation class to accept more graphics types
-------------------------------------+-------------------------------------
       Reporter:  niles              |        Owner:  jason, was
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.1
      Component:  graphics           |   Resolution:
       Keywords:  animate,           |    Merged in:
  graphics, 3D                       |    Reviewers:
        Authors:  Niles Johnson      |  Work issues:  docstrings, testing,
Report Upstream:  N/A                |  think about img protocol
         Branch:                     |       Commit:
  u/niles/ticket/12827               |  de5349040ebab566d5a0e35d99284e628ab21080
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by jhpalmieri):

 First, overall it looks pretty good to me.

 Second, a typo: on the third line of `animate.py`, "iteratable" should be
 "iterable".

 Third, I'm having problems with ffmpeg; this is on OS X 10.9. To try to
 fix them, I installed the latest version (from the git repo) of ffmpeg,
 but the problems persist.

 - the `-loop_output` flag leads to the error `Unrecognized option
 'loop_output'.` The man page says
 {{{
        -loop_output number_of_times
            Repeatedly loop output for formats that support looping such as
            animated GIF (0 will loop the output infinitely).  This option
 is
            deprecated, use -loop.
 }}}
  Of course, the man page doesn't say anything at all about `-loop` so I
 don't know exactly how to use it, but just changing `-loop_output` to
 `-loop` produces a valid animated file. (I also don't know how long the
 `-loop` option has been available, so if we switch to that, will we just
 be breaking old versions of ffmpeg, or will it affect fairly recent ones?)

 - the `-g 3` option leads to `Codec AVOption g (set the group of picture
 (GOP) size) specified for input file #0 (/Users/palmieri/.sage/temp
 /Macintosh-001b639d44a1.local/67838/dir_lolwRD/%08d.png) is not a decoding
 option.` Removing this option completely works. (I also can't find it in
 the man page, so I don't know what it's supposed to do. The man page is
 really not very good.)

 - the `-pix_fmt rgb24` option leads to `Incompatible pixel format 'rgb24'
 for codec 'gif', auto-selecting format 'pal8'`. I just deleted this option
 and it works. Should we do that, or use `-pix_fmt rgb8` (for example)
 instead? The available formats may depend on how ffmpeg was installed, so
 it might be better to omit it and allow users to specify it using the
 `ffmpeg_options` argument.

 - finally, the gif file produced by ffmpeg (for example using the `sines`
 animation) looks pretty bad: the axes and sine waves are multi-colored,
 and not in a good way. Using ffmpeg to produce a .mov file or .avi file
 works well.

--
Ticket URL: <http://trac.sagemath.org/ticket/12827#comment:19>
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/groups/opt_out.

Reply via email to