Skip to content

Commit

Permalink
Merge pull request nrkno#1266 from bbc/fix/autonext-postroll
Browse files Browse the repository at this point in the history
Bug fix: currentPart timeline duration should respect postroll in autonext
  • Loading branch information
jstarpl authored Sep 26, 2024
2 parents 31d94a8 + 7fe60ea commit 5bc0155
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/job-worker/src/playout/timeline/piece.ts
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ export function getPieceEnableInsidePart(
if (partTimings.toPartPostroll) {
if (!pieceEnable.duration) {
// make sure that the control object is shortened correctly
pieceEnable.duration = `#${partGroupId} - ${partTimings.toPartPostroll}`
pieceEnable.end = `#${partGroupId} - ${partTimings.toPartPostroll}`
}
}

Expand Down
3 changes: 2 additions & 1 deletion packages/job-worker/src/playout/timeline/rundown.ts
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,8 @@ export function buildTimelineObjsForRundown(
// If there is a valid autonext out of the current part, then calculate the duration
currentPartEnable.duration =
partInstancesInfo.current.partInstance.part.expectedDuration +
partInstancesInfo.current.calculatedTimings.toPartDelay
partInstancesInfo.current.calculatedTimings.toPartDelay +
partInstancesInfo.current.calculatedTimings.toPartPostroll // autonext should have the postroll added to it to not confuse the timeline
}
const currentPartGroup = createPartGroup(partInstancesInfo.current.partInstance, currentPartEnable)

Expand Down

0 comments on commit 5bc0155

Please sign in to comment.