<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <p><br>

    </p>

    <br>

    <div class="moz-cite-prefix">On 07/16/2018 11:37 AM, David Engel

      wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:20180716153702.GA26844@opus.istwok.net">

      <blockquote type="cite" style="color: #000000;">

        <pre wrap="">I already have a flag stored in the decoder and accessed from mythplayer

using "decoder->GetfpsMultiplier()" It takes values 1 or 2, 1=normal,

2=doubled frames as with mediacodec.

</pre>

      </blockquote>

      <pre wrap="">Excellent!

</pre>

      <blockquote type="cite" style="color: #000000;">

        <pre wrap="">I am currently using fpsMultiplier only in AVSync where frame_delay is

calculated from frame_interval. I need to take that out and change the

calculation of frame_interval itself to use fpsMultiplier instead.

</pre>

      </blockquote>

      <pre wrap="">So are you going to make the avsync_predictor_enabled change then or

do you want me to do it?  I have time for it today.

</pre>

    </blockquote>

    I have created a new patch that changes frame_interval calculation,

    and attached it to the ticket. I also set the buffers to 8 instead

    of 4. With 4 buffers there still was a slight jiggle in the 2x

    speedup playback, that is gone with 8 buffers. See what you think.<br>

    <br>

    FF seems to be working OK with this patch. The failure is rather

    random. The re-init handles it with a slight pause when it does

    happen. Do you think there is any point in trying to change FF to

    use software decoding?<br>

    <br>

    I think I am almost ready to commit this. First I will do some

    testing on Linux with different playback profiles, and on raspberry

    Pi, to make sure nothing got broken. I want to split it into a few

    commits to separate some things that were not exclusively for

    mediacodec, and separate out the FFmpeg fix. Anything that is

    logically separate I will make a separate commit. I see no sign that

    Aman plans to commit the FFmpeg fix. That needs to be committed to

    the MythTV private FFmpeg repository as well.<br>

    <br>

    Peter<br>

  </body>

</html>