Opened 20 years ago

Closed 20 years ago

Last modified 20 years ago

#237 closed patch (fixed)

Regenerate keyframe index for MPEG2->MPEG2 transcoding

Reported by: Aaron McCarthy <mccarthy.aaron@…> Owned by: jdonavan
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: medium Keywords: MPEG2 transcoding
Cc: Ticket locked: no

Description

Two problems.

After transcoding mpeg2 recordings the regenerated keyframe index is immediately deleted by the jobqueue code.

Secondly, on some setups the recordedmarkup table contains MARK_GOP_START entries instead of MARK_GOP_BYFRAME. The keyframe index is regenerated if MARK_GOP_BYFRAME entries are not found. This is required on at least one PVRx50 setup.

Attachments (1)

mpeg2trans.patch (3.0 KB ) - added by Aaron McCarthy <mccarthy.aaron@…> 20 years ago.

Download all attachments as: .zip

Change History (4)

by Aaron McCarthy <mccarthy.aaron@…>, 20 years ago

Attachment: mpeg2trans.patch added

comment:1 by Isaac Richards, 20 years ago

Owner: changed from Isaac Richards to jdonavan

comment:2 by cpinkham, 20 years ago

Resolution: fixed
Status: newclosed

(In [7263]) JobQueue, mythtranscode, and logging related modifications.

Main patches by Robert Tsai & Aaron McCarthy with a little merging by me.

  • Add logging to display mythtranscode's filespace savings.
  • Logging cleanup and addition of subtitle (if present) in some places.
  • Regenerate MARK_GOP_BYFRAME index on recordings that only have MARK_GOP_START since the transcoder needs MARK_GOP_BYFRAME.
  • Cleanup old keyframe/gop index in mythtranscode rather than having the JobQueue do it. The JobQueue has no way of knowing whether mythtranscode generated a MPEG2 file and corresponding MARK_GOP_BYFRAME entries or if it generated a MPEG4/RTJPEG file with the seektable in the nuppel file, so it only makes sense to move this logic into mythtranscode.

Closes #325 and closes #237.

comment:3 by cpinkham, 20 years ago

(In [7264]) Transcoding patch (for real this time).

This is part of Aaron McCarthy's patch that I forgot to apply as part of my last commit. This commit should really close #237.

Note: See TracTickets for help on using tickets.