#7298: use html5 video tag for animations
------------------------------+---------------------------------------------
Reporter: whuss | Owner: whuss
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.6.1
Component: graphics | Resolution: fixed
Keywords: animation, video | Author: Wilfried Huss
Upstream: N/A | Reviewer: Pablo Angulo, Niles Johnson
Merged: | Work_issues: pass list rather than single
string to ``check_call``
------------------------------+---------------------------------------------
Changes (by niles):
* status: needs_review => needs_work
* reviewer: Pablo Angulo => Pablo Angulo, Niles Johnson
* work_issues: => pass list rather than single string to ``check_call``
Comment:
Replying to [comment:13 niles]:
> Replying to [comment:12 whuss]:
> > Replying to [comment:11 niles]:
> Thanks; the problem seems to be that, for me, `check_call` needs to be
called with a list of inputs -- giving the command as a single string
fails for some reason (`png2theora` returns its usage message, so probably
it is not receiving the arguments in the expected format):
>
{{{
sage: check_call(["png2theora", '-o "tmp2.ogv"', '%08d.png'])
9 frames, 202x106
Compressing....
./00000000.png
./00000001.png
./00000002.png
./00000003.png
./00000004.png
./00000005.png
./00000006.png
./00000007.png
./00000008.png
done.
0
}}}
>
> but
{{{
sage: check_call(["png2theora", '-o "tmp2.ogv" %08d.png'])
png2theora 1.1
Usage: png2theora [options] <input>
...
}}}
>
> and
{{{
sage: check_call('png2theora -o "tmp3.ogv" %08d.png')
---------------------------------------------------------------------------
OSError Traceback (most recent call
last)
...
OSError: [Errno 2] No such file or directory
}}}
>
> The last command is a simplified version of what appears in
`sage/plot/animate.py` . . . does it really work for other people here?
And if so, why?
This seems to be the offending code:
{{{
405 cmd = 'cd "%s"; sage-native-execute png2theora -o "%s" -f
%s %%08d.png 2> /dev/null'%(d, savefile, int(100/delay))
406 from subprocess import check_call, CalledProcessError
407 try:
408 check_call(cmd, shell=True)
}}}
I'm switching this to "needs work", since I think ``cmd`` needs to be a
list of strings rather than a single string (see above).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7298#comment:16>
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 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.
