Last modified: 2010-05-04 01:34:18 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T25160, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 23160 - Theora video thumbnails corrupt
Theora video thumbnails corrupt
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
PC Windows Vista
: Normal major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 22100 23163 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-04-11 22:39 UTC by 84user
Modified: 2010-05-04 01:34 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Test_20100411_Vdubtestrgbcube4secs_ifps.ogv with empty thumbnail (29.78 KB, image/png)
2010-04-11 22:39 UTC, 84user
Details
Debug output (6.92 KB, text/plain)
2010-04-14 08:51 UTC, Roan Kattouw
Details

Description 84user 2010-04-11 22:39:43 UTC
Created attachment 7285 [details]
Test_20100411_Vdubtestrgbcube4secs_ifps.ogv with empty thumbnail

Between 03:23, 8 April 2010 and 08:40, 9 April 2010 new video uploads started to render with empty or corrupt thumbnails (Firefox, Google Chrome, Opera and Safari, with cache and file purging; thumbtime also tested). New uploads are still rendering incorrectly. I *believe* I have eliminated file corruption because the file uploaded is identical to the file on my PC (I download the file from mediawiki and use emacs to compare).

See http://commons.wikimedia.org/wiki/Commons:Village_pump#Video_thumbnails_corrupt_since_9_April

Duplicated problem on skins: Monobook, Vector and Classic. Other skins not tried.

How to replicate:

1: choose any small video from Category:Calibration videos at http://commons.wikimedia.org/wiki/Category:Calibration_videos and download to computer

2: upload same file with name prefixed with "Test_reupload_"; click the "ignore warning and save anyway" button

3: see the corrupt or empty video thumbnail

Example attached from: http://commons.wikimedia.org/wiki/File:Test_20100411_Vdubtestrgbcube4secs_ifps12fr12_q10.ogv
Comment 1 Derk-Jan Hartman 2010-04-11 23:26:48 UTC
The empty thumbs are due to the fact that there are no key frames following the seek point (midway file). We should probably always start at frame 0 when a movie is under 10seconds.

The other issue is of a similar nature and also related to keyframes.

What triggered this behavior, i'm not sure of, but most likely ffmpeg was upgraded on the scalers. I have asked Roan, and the servers currently run r11872+debian_3:0.svn20080206-12ubuntu

If someone could run the following on a scaling server and extract the debug info from ffmpeg please. That might help:

ffmpeg -ss 9.5 -i /mnt/upload6/wikipedia/commons/b/bd/Translational_motion_gif.ogv -f mjpeg -an -vframes 1 ./test.jpg

I made an educated guess at the path. Might not be 100% correct.
Comment 2 Derk-Jan Hartman 2010-04-11 23:42:51 UTC
I note that wmf-deployment has some OggHandler/OggHandler_body.php changes from mdale, that are not currently in wmf/1.16wmf4
Comment 3 Derk-Jan Hartman 2010-04-12 00:09:43 UTC
*** Bug 23163 has been marked as a duplicate of this bug. ***
Comment 4 Sage Ross 2010-04-12 17:46:43 UTC
Is there a workaround, or are all thumbnails on new videos broken for now?
Comment 5 Gregory Maxwell 2010-04-12 21:42:25 UTC
The thumbnails for the files linked in in Bug 23163 don't really look like failing to decode starting at a keyframe. They look like the result of decoding with the broken theora encoder which was included in ffmpeg up until about a year ago.

At least of the cluster was upgraded to fix this at some point, but perhaps some old software is still out there on some systems.
Comment 6 Derk-Jan Hartman 2010-04-12 23:34:13 UTC
(In reply to comment #4)
> Is there a workaround, or are all thumbnails on new videos broken for now?

there is currently no workaround. When everyone has landed in Berlin, Tim Starling and/or Michael Dale will have to look at it probably.
Comment 7 Gregory Maxwell 2010-04-12 23:46:39 UTC
(In reply to comment #5)
> with the broken theora encoder which was included in ffmpeg up until about a

*decoder*   ugh.


http://lists.wikimedia.org/pipermail/wikitech-l/2010-February/046706.html  < this thread is likely relevant too.
Comment 8 Derk-Jan Hartman 2010-04-12 23:51:26 UTC
(In reply to comment #7)
> http://lists.wikimedia.org/pipermail/wikitech-l/2010-February/046706.html  <
> this thread is likely relevant too.

Yes, that's exactly why I pointed out the differences between wmf-deployment and 1.16wmf4
Comment 9 Roan Kattouw 2010-04-14 08:51:46 UTC
Created attachment 7290 [details]
Debug output

Debug output as requested in comment #1.
Comment 10 Gregory Maxwell 2010-04-14 12:51:41 UTC
The ffmpeg in that debugging output is too old. Even absent the seeking problems it will not reliably produce correct output.
Comment 11 Michael Dale 2010-04-15 16:29:53 UTC
there is also this code review comment from Tim that says that oggThumb is in fact oky? 
http://www.mediawiki.org/wiki/Special:Code/MediaWiki/62223#c5744

... but would have to go into the "code review queue" ? But the fix was already in the deployment branch without any suggested changes. So Tim forgot to merge it before he deployed it ?  

I can send a patch to make wmf4 what was once wmf-deployment adding in Tim's fix to bug 22388 ? Does that sound reasonable? 

And we the other parts of oggHandler can go back into trunk for review ? ( i.e transcoding so have some basic consistency in the bitrates of embed videos ? )
Comment 12 Michael Dale 2010-04-17 00:28:26 UTC
Restored the previously deployed ( working ) oggThumb code to oggHandler trunk. r65149 

Its kind of tricky to test since you need the svn version of oggThumb ( that has already been deployed )
Comment 13 Derk-Jan Hartman 2010-04-18 15:26:24 UTC
*** Bug 22100 has been marked as a duplicate of this bug. ***
Comment 14 Michael Dale 2010-04-21 22:26:27 UTC
any update on this? 

Almost all newly uploaded videos continue to have broken thumbnails ... 

Should I be giving opts people instruction to compile ffmpeg to deploy less broken thumbnailing?
Or can we put the code that was working for a few months back into operation while this gets sorted?
Comment 15 Gregory Maxwell 2010-04-21 22:36:12 UTC
FWIW— Yet another Ffmpeg-violates-the-spec causing failed decoding problem was discovered and fixed today.   This isn't a case that people should have been hitting with any frequency but it's a reason to use only the most current development ffmpeg if ffmpeg is continued to be used.
Comment 16 84user 2010-04-28 20:03:17 UTC
FWIW2- ffmpeg2theora version 0.24 is giving good thumbnails for me lately. Versions 0.25, 0.26 and from 2010-03-16 always gives garbled or missing thumbnails. For small examples see [http://commons.wikimedia.org/wiki/File:SDO 10 second youtube 1080 pixels DA8C8E04d01.flv 320x240 br270.ogv] and  [http://commons.wikimedia.org/wiki/File:Solar Dynamics Observatory xo with ffmpeg2theora-0.24 320x240 br270.ogv]: scroll down to "Source" for the exact commands used to encode. I also posted this at [http://commons.wikimedia.org/wiki/Help_talk:Converting_video#Video_thumbnail_error] and [http://commons.wikimedia.org/wiki/Commons:Village_pump#Theora_thumbnails].
Comment 17 84user 2010-04-28 20:07:08 UTC
(In reply to comment #16)
> FWIW2- ffmpeg2theora version 0.24 is giving good thumbnails for me lately.
> Versions 0.25, 0.26 and from 2010-03-16 always gives garbled or missing
> thumbnails. For small examples see [http://commons.wikimedia.org/wiki/File:SDO
> 10 second youtube 1080 pixels DA8C8E04d01.flv 320x240 br270.ogv] and 
> [http://commons.wikimedia.org/wiki/File:Solar Dynamics Observatory xo with
> ffmpeg2theora-0.24 320x240 br270.ogv]: scroll down to "Source" for the exact
> commands used to encode. I also posted this at
> [http://commons.wikimedia.org/wiki/Help_talk:Converting_video#Video_thumbnail_error]
> and [http://commons.wikimedia.org/wiki/Commons:Village_pump#Theora_thumbnails].

The above URLs should read:

http://commons.wikimedia.org/wiki/File:SDO_10_second_youtube_1080_pixels_DA8C8E04d01.flv_320x240_br270.ogv

and

http://commons.wikimedia.org/wiki/File:Solar_Dynamics_Observatory_xo_with_ffmpeg2theora-0.24_320x240_br270.ogv

also

http://commons.wikimedia.org/wiki/File:Helioseismic_and_Magnetic_Imager_First_Light_Briefing_320x180_ffmpeg2theora-0.24.ogv
Comment 18 Tim Starling 2010-04-30 00:43:17 UTC
I've rewritten the oggThumb support in OggHandler and I've made some substantial changes to oggThumb itself in a development branch on sf.net. I'm going to try handing this over to Mark Bergsma for packaging.
Comment 19 Tim Starling 2010-05-04 01:34:18 UTC
Fixed now. Any remaining cases should be fixable by doing action=purge on the image description page followed by forced refresh (e.g. shift-reload in Firefox).

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links