Skip to content

Conversation

@rjmunro
Copy link

@rjmunro rjmunro commented Nov 17, 2025

About the Contributor

This pull request is posted on behalf of the BBC.

Type of Contribution

This is a Feature

Current Behavior

Currently it's difficult to combine an AB session for the M/E input with AB sessions on the Upstream Keyers due to them being handled in the same timeline object.

New Behavior

Allow addressing each keyer on each M/E individually as a separate layer in TSR.

Backwards compatibility is provided; you will get a one of warning per session telling you to upgrade. If you use both the backwards compatibility and the new method at the same time, you should get a further warning and the new method should win.

Testing Instructions

I've tested with some scripts in QuickTSR and it seems to be working. To test it properly you will need to rewrite blueprints to use it.

Other Information

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

@rjmunro rjmunro marked this pull request as ready for review November 20, 2025 15:35
@rjmunro rjmunro requested a review from a team as a code owner November 20, 2025 15:35
Copy link
Member

@Julusian Julusian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of questions/thoughts

@rjmunro rjmunro force-pushed the rjmunro/separate-upstream-keyer branch 2 times, most recently from 4c49064 to 7102a19 Compare December 1, 2025 21:03
@rjmunro rjmunro force-pushed the rjmunro/separate-upstream-keyer branch from 7102a19 to 2b21d32 Compare December 3, 2025 14:34
@Saftret Saftret added the Contribution from BBC Contributions sponsored by BBC (bbc.co.uk) label Dec 10, 2025
@Julusian Julusian merged commit dec93f7 into release53 Dec 17, 2025
39 checks passed
@rjmunro rjmunro deleted the rjmunro/separate-upstream-keyer branch December 17, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contribution from BBC Contributions sponsored by BBC (bbc.co.uk)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants