Modify

Ticket #10872 (closed Bug Report - General: fixed)

Opened 11 months ago

Last modified 10 months ago

Recording Quality score invalid for intentional late starts, early ends

Reported by: danielk Owned by: danielk
Priority: blocker Milestone: 0.26
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

When a user intentionally sets up a recording to start late or end early it gets marked as having poor quality and the recording is eligible for re-recording. Instead we should account for this in the RecordingQuality?() constructor and not count these start/end "gaps" against the recording.

Attachments

10872-v2.patch (13.0 KB) - added by danielk 11 months ago.

Change History

comment:1 Changed 11 months ago by danielk

  • Owner set to danielk
  • Status changed from new to accepted

comment:2 Changed 11 months ago by gigem

Daniel,

Stopping a recording early should also not count against the recording quality score.

Changed 11 months ago by danielk

comment:3 Changed 11 months ago by danielk

The idea in this patch is to track the intended recording start and end times in RecordingInfo?. So far this captures the start and end times from the scheduler, but we still need to update the desired recording end time when the user ends the recording early. And of course we need to test thoroughly once complete.

comment:4 Changed 11 months ago by danielk

  • Priority changed from minor to blocker

comment:5 Changed 10 months ago by Daniel Thor Kristjansson <danielk@…>

  • Status changed from accepted to closed
  • Resolution set to fixed

In 07acf0e850d62d6c54cec498c675cc8f82d83d64/mythtv:

Fixes #10872. Use desired start and end times.

The recording quality measurement needs to know when a recording was
supposed to start and end to know how much is actually missing from
the recording. We used to use the scheduled start and end times for
this, but sometimes a recording is intentionally started late or ended
early. This captures that by tracking the 'desired' start and end point.
This is initially set by the scheduler using and pre and post roll
values, and if the user intentionally stops a recording early that
updates the desired end time as well.

comment:6 Changed 10 months ago by danielk

  • Status changed from closed to new
  • Resolution fixed deleted

This shouldn't have been marked fixed yet.

comment:7 Changed 10 months ago by Daniel Thor Kristjansson <dkristjansson@…>

In dc09143852e560266e8454ed9854705c87f330b9/mythtv:

Refs #10872. Use desired start and end times for recording quality.

This is the rest of [07acf0e85]. The constructors weren't right in
the previous commit and we need to get updated desired times to
the recorder for the recording quality determination to use them.

comment:8 Changed 10 months ago by Daniel Thor Kristjansson <danielk@…>

  • Status changed from new to closed
  • Resolution set to fixed

In ecc1525d83abfed452e3b5f983de7c593063af91/mythtv:

Fixes #10872. Use desired start and end times.

The recording quality measurement needs to know when a recording was
supposed to start and end to know how much is actually missing from
the recording. We used to use the scheduled start and end times for
this, but sometimes a recording is intentionally started late or ended
early. This captures that by tracking the 'desired' start and end point.
This is initially set by the scheduler using and pre and post roll
values, and if the user intentionally stops a recording early that
updates the desired end time as well.

Conflicts:

mythtv/libs/libmythtv/dtvrecorder.h

comment:9 Changed 10 months ago by Daniel Thor Kristjansson <dkristjansson@…>

In 16231c959b2edbef011636e3c8df6e66933899e9/mythtv:

Refs #10872. Use desired start and end times for recording quality.

This is the rest of [07acf0e85]. The constructors weren't right in
the previous commit and we need to get updated desired times to
the recorder for the recording quality determination to use them.

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.