[MPlayer-dev-eng] [PATCH] new mpeg muxer

[D Richard Felker III]

On Thu, Feb 24, 2005 at 05:22:42PM +0100, Nico Sabbi wrote:
> D Richard Felker III wrote:
> 
> >On Thu, Feb 24, 2005 at 04:01:13PM +0100, Nico Sabbi wrote:
> > 
> >
> >>Tonight I will disable frame reordering by default,  and try to fix what 
> >>causes the flashing
> >>when scene change kicks in. What value did you use for sc_threshold?
> >>   
> >>
> >
> >before i was using whatever the default is, but i changed it to
> >1000000 to disable scene change detection. actually it needs to be
> >disabled for compliant gop structure, i think...but still the muxer
> >shouldn't misorder frames.
> >
> >rich
> >
> > 
> >
> 
> this really puzzles me, I have never seen the muxer misordering the frames,
> and I believe the code is correct.
> Do you see the same effect if you play the dumped video?

huh? what do you mean by play the dumped video? what i'm doing is
encoding to mpeg with a single run of mencoder, using lavc's
mpeg1video codec with b frames. i definitely see the frames get
decoded wrong (a b-frame comes out looking like complete nonsense) at
points where lavc is inserting a new i-frame for scene change when i
play the resulting file with mplayer. on the hardware vcd player i
don't recall seeing any such glitches, probably because it throws away
the frame because the way it's ordered makes no sense...

rich