Opened 14 years ago

Closed 14 years ago

#325 closed patch (fixed)

Log transcoding filespace savings

Reported by: Robert Tsai <rtsai1111> Owned by: Isaac Richards
Priority: minor Milestone: 0.19
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

This patch adds some logging to record mythtranscode filespace savings. In the database, it adds (pasted from mythweb):

4622  	transcode  	2005-09-11 18:41:22  	myth  	Transcode Finished  	Secrets of the Dead "Mystery of the Black Death": 1.9 GB (Medium Quality)
4621 	transcode 	2005-09-11 17:00:15 	myth 	Transcode Starting 	Secrets of the Dead "Mystery of the Black Death": 6.0 GB (Medium Quality)

In the jobqueue table, it adds the following text, which will appear on the mythweb backend status page (pasted from mysql):

| 963 |    1 |    0 |    272 | Low Quality: 7.8 GB => 1.6 GB      |
| 964 |    2 |    0 |    272 | Finished, 1 break(s) found.        |
| 965 |    1 |    0 |    272 | Medium Quality: 6.0 GB => 1.9 GB   |
+-----+------+------+--------+------------------------------------+

It also does a little bit of logging cleanup (removing redundantly-logged information).

Attachments (3)

mythtv.logging.diff (11.4 KB) - added by Robert Tsai <rtsai1111> 14 years ago.
mythtv.logging.2.diff (11.3 KB) - added by Robert Tsai <rtsai1111> 14 years ago.
updated diff against r7255
mythtv.logging.3.diff (709 bytes) - added by Robert Tsai <rtsai1111> 14 years ago.
fix typo in r7263

Download all attachments as: .zip

Change History (6)

Changed 14 years ago by Robert Tsai <rtsai1111>

Attachment: mythtv.logging.diff added

Changed 14 years ago by Robert Tsai <rtsai1111>

Attachment: mythtv.logging.2.diff added

updated diff against r7255

comment:1 Changed 14 years ago by cpinkham

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:2 Changed 14 years ago by Robert Tsai <rtsai1111>

Milestone: 0.19
Resolution: fixed
Status: closedreopened

There is a small typo in r7263. Patch attached.

Changed 14 years ago by Robert Tsai <rtsai1111>

Attachment: mythtv.logging.3.diff added

fix typo in r7263

comment:3 Changed 14 years ago by cpinkham

Resolution: fixed
Status: reopenedclosed

(In [7265]) Patch by Robert Tsai to fix a typo in my previous merge. (Re) closes #325.

Note: See TracTickets for help on using tickets.