libvpx does not support timebase > 1

[GoogleCodeExporter]

libvpx requires timebase numerator to be smaller than denominator, so its 
magnitude must be < 1. It returns an error on encoder init otherwise.

There should be no reason for this limitation, setting the timebase such that 
its magnitude is > 1 is perfectly sane for streams with less than 1 fps.

Original issue reported on code.google.com by anton.kh...@gmail.com on 5 Oct 2012 at 6:05

[GoogleCodeExporter]

Original comment by albe...@google.com on 11 Oct 2012 at 10:10

• Changed state: Available
• Added labels: Mstone-14, Pri-3
• Removed labels: Pri-2

[GoogleCodeExporter]

I was able to change one line (libvpx-1.2.0) in vp8/vp8_cx_iface.c

<    RANGE_CHECK(cfg, g_timebase.num,        1, cfg->g_timebase.den);
-
>    RANGE_CHECK(cfg, g_timebase.num,        1, 1000000000);

Works great for me.  The webm overview claims no limitations on frame rate- but 
there could be another reason this is this way.   

I'll look into submitting this as a patch after Christmas.

-Johnny Klonaris

Original comment by yout...@jawknee.com on 21 Dec 2013 at 11:54

[GoogleCodeExporter]

Was this patch ever submitted? 1.5 years later I have just hit this problem. We 
generate a number of different formats and it is the webm that fails because of 
< 1 second values. 

Original comment by justinis...@gmail.com on 1 Jun 2015 at 4:02

[GoogleCodeExporter]

typo:  s/second/fps

Original comment by justinis...@gmail.com on 1 Jun 2015 at 4:03

[GoogleCodeExporter]

No, the patch wasn't - and I need to do that.    I might want some help testing 
on Windows.

Original comment by goo...@jawknee.com on 1 Jun 2015 at 4:05

[GoogleCodeExporter]

I've submitted the patch - we'll see if it flies.

Original comment by goo...@jawknee.com on 1 Jun 2015 at 7:28

[GoogleCodeExporter]

Thanks for following up on that! 

Original comment by justinis...@gmail.com on 1 Jun 2015 at 8:03

[GoogleCodeExporter]

https://chromium-review.googlesource.com/#/c/274107/

Seems like the CI failed because of "committer/uploader not found."

Original comment by justinis...@gmail.com on 1 Jun 2015 at 10:59

[GoogleCodeExporter]

Original comment by johannkoenig@google.com on 1 Jun 2015 at 4:25