kim42083 / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

Cues improperly supported by libwebm #383

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.  Use libwebm to create a simple video file using AddFrame()
2.  Use webminspector.py to review the file

What is the expected output? What do you see instead?
-Expected to see Cues before Cluster with CuePoints for each video key frame 
(in accordance with Muxer Guidelines at webmproject.org)
-Instead, a Seek exists for a non-existent Cues at end of file.

What version are you using? On what Windows version?
libwebm 1.0.0.5

Please provide any additional information below.
Disabling Cues output via OutputCues(false) still causes a Seek element to be 
emitted for a non-existant Cues element.  Either support Cues properly or 
support disabling Cues properly.

Original issue reported on code.google.com by kipt...@gmail.com on 5 Jan 2012 at 7:45

GoogleCodeExporter commented 9 years ago

Original comment by iss...@webmproject.org on 5 Jan 2012 at 7:50

GoogleCodeExporter commented 9 years ago
Version is a snapshot from trunk dated JAN-2-2012 (mistakenly used version from 
RELEASE.TXT).

Original comment by kipt...@gmail.com on 5 Jan 2012 at 8:01

GoogleCodeExporter commented 9 years ago
It could be a bug on the inspector. 

Original comment by albe...@google.com on 16 Feb 2012 at 9:10

GoogleCodeExporter commented 9 years ago

Original comment by albe...@google.com on 16 Feb 2012 at 9:11

GoogleCodeExporter commented 9 years ago
I fixed the Disable Cues output via OutputCues(false).

https://gerrit.chromium.org/gerrit/#change,16064

Original comment by fgalli...@google.com on 16 Feb 2012 at 9:47

GoogleCodeExporter commented 9 years ago
I cannot reproduce "-Instead, a Seek exists for a non-existent Cues at end of 
file." I always get a Cues at the end of the file when I ask for it. Can you 
post more information?

Original comment by fgalli...@google.com on 16 Feb 2012 at 9:49

GoogleCodeExporter commented 9 years ago
As for the guidelines, they need to be changed. Most tools and clients expect 
the Cues at the end of the file.

Original comment by fgalli...@google.com on 16 Feb 2012 at 9:50

GoogleCodeExporter commented 9 years ago
Disabling cues works correctly now, thank you.

Enabling cues looks correct for longer videos.  For a very short video (1 
Cluster), webminspector reports a Seek element for Cues.  But, webminspector 
shows no Cues, Cue Points or Cue Track Elements (there should be 1 set).  With 
a longer video (7 Clusters), webminspector shows all Cueing elements correctly.

Maybe a 2 Cluster assumption error in the libwebm code?  Maybe in the 
webminspector code?

The guidelines on Cueing at WebM go so far as to link to a tool at Matroska for 
moving Cues from the end of the file to the beginning of the file.  It seems 
like someone had a strong case for having Cues at the beginning of the file.  
But, I don't  know what most WebM tools are actually doing.  Whatever works and 
is consistently documented is fine with me.

Original comment by kipt...@gmail.com on 17 Feb 2012 at 9:59

GoogleCodeExporter commented 9 years ago

Original comment by albe...@google.com on 8 Mar 2012 at 12:26

GoogleCodeExporter commented 9 years ago
kiptonk can we get a sample of your one cluster file?

Original comment by fgalli...@google.com on 24 May 2012 at 10:03

GoogleCodeExporter commented 9 years ago
One cluster file attached.

Original comment by kipt...@gmail.com on 29 May 2012 at 9:51

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by fgalli...@google.com on 9 Oct 2014 at 11:40