Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send REC_PENDING event before recording starts #716

Merged
merged 2 commits into from
Feb 11, 2023
Merged

Send REC_PENDING event before recording starts #716

merged 2 commits into from
Feb 11, 2023

Conversation

kmdewaal
Copy link
Contributor

@kmdewaal kmdewaal commented Feb 9, 2023

Fix the logic that sends the REC_PENDING event.

  • REC_PENDING event is sent always before the REC_STARTED.
  • REC_PENDING event is sent at 120, 90, 60 and 30 seconds before the start of the recording.
  • Start of the recording is the scheduled program start time plus the RecordPreRoll setting value.
  • When a recording is scheduled to start recording immediately then there is
    one REC_PENDING event with 0 seconds immediately followed by the REC_STARTED event.
  • The setting WakeUpThreshold specifies how early the slave backends are signaled to start
    before the recording is started. This setting is used as configured, but now with a minimum
    value of RecordPreRoll plus 120 seconds so that there is always time for the REC_PENDING events.

Fix the logic that sends the REC_PENDING event.
- REC_PENDING event is sent always before the REC_STARTED.
- REC_PENDING event is sent at 120, 90, 60 and 30 seconds before the start of the recording.
- Start of the recording is the scheduled program start time plus the RecordPreRoll setting value.
- When a recording is scheduled to start recording immediately then there is
  one REC_PENDING event with 0 seconds immediately followed by the REC_STARTED event.
- The setting WakeUpThreshold specifies how early the slave backends are signaled to start
  before the recording is started. This setting is used as configured, but now with a minimum
  value of RecordPreRoll plus 120 seconds so that there is always time for the REC_PENDING events.
@kmdewaal kmdewaal requested review from gigem and paul-h February 9, 2023 21:17
@kmdewaal kmdewaal self-assigned this Feb 9, 2023
@kmdewaal kmdewaal merged commit c9c1417 into MythTV:master Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant