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

Requests: Add inputActive field to GetInputList response #1023

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

Conversation

awfulcooking
Copy link

@awfulcooking awfulcooking commented Sep 19, 2022

Description

Adds inputActive bool to objects in the GetInputList response

Motivation and Context

There isn't currently a way to determine which inputs are active, without hacks(*)

This field makes loading the state much easier. So we can show only active inputs in an audio mixer, like the OBS frontend does.

(* Only active inputs are sent in the high frequency InputVolumeMeters event, so you can filter based on an input's presence in that)

How Has This Been Tested?

Tested OS(s): compiles in a Linux container ☑️

Can't easily test further atm.

Checklist:

  • I have read the Contributing Guidelines.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • My code is not on master or a release/* branch.
  • The code has been tested.
  • I have included updates to all appropriate documentation.

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.

1 participant