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

Add RFC: Create output manager #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cg2121
Copy link
Contributor

@cg2121 cg2121 commented Sep 28, 2022

Description

Add an output manager so that outputs can be managed in a single dialog.

Motivation and Context

  • Each output (NDI, Decklink, AJA, etc.) doesn't have to write their own UI, removing much deplicated code.
  • It would make it easier to find bugs and fix them for all outputs instead of just one.

Link

https://github.com/cg2121/rfcs/blob/output-manager-rfc/text/0052-output-manager.md

Add an output manager so that outputs can be managed in a single dialog.
@Fenrirthviti
Copy link
Member

This seems like it would be more complex and extra steps for users to have to manage their non-stream/recording outputs. Many outputs are trying to offer different thing, so there isn't much (at least, as I see it) that would be really shared through this. Unless I'm misunderstanding, it would add an additional step where a user has to select Tools -> Output Manager -> NDI/Decklink/etc.

I think screenshots or mockups of what this would look like, and a better explanation on what code would be simplified would help here. If there's something lacking from OBS to creating these custom outputs properties, would it make more sense to handle this as purely a backend/plugin API problem instead of trying to bolt everything to an outputs manager?

@Fenrirthviti
Copy link
Member

As there has been no response or clarification on this, moving to FCP with disposition close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants