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

Fix audio only sources #350

Merged
merged 3 commits into from
Nov 9, 2024
Merged

Fix audio only sources #350

merged 3 commits into from
Nov 9, 2024

Conversation

justjanne
Copy link
Contributor

  • Alsa should be included in the audio only source list
  • Don't try to mirror video for audio only sources
  • VoctoGUI relies on VideoDisplay to update the audio levels, create an audio only alternative

@justjanne justjanne force-pushed the fix-audio-only-sources branch from bb340f5 to 997fa29 Compare November 2, 2024 17:44
Copy link
Contributor

@Kunsi Kunsi left a comment

Choose a reason for hiding this comment

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

Thanks! Do you have a screenshot of how this looks in voctogui?


def mute(self, mute):
self.pipeline.get_by_name("audiosink-{name}".format(name=self.name)).set_property(
"volume", 1 if mute else 0)
Copy link
Contributor

@Kunsi Kunsi Nov 2, 2024

Choose a reason for hiding this comment

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

Great, this is inverted (but also inverted in videodisplay.py, so let's keep this here as well).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tbh, this is just a quick workaround - a better long-term solution would probably be for videodisplay to handle optional audio or optional video itself.

@justjanne
Copy link
Contributor Author

Thanks! Do you have a screenshot of how this looks in voctogui?

The audioonlysource doesn't show up at all in the video sources on the left, but it does show up in the audio tab on the bottom. It looks just like you'd expect.

I'll try to set up the encoder in the next few days again, then I'll be able to attach a screenshot as well.

@Kunsi Kunsi merged commit f829874 into voc:main Nov 9, 2024
@Kunsi
Copy link
Contributor

Kunsi commented Nov 23, 2024

Atleast for this configuration, livesources=slides is audio-only:

[mix]
videocaps=video/x-raw,format=I420,width=1920,height=1080,framerate=25/1,pixel-aspect-ratio=1/1,interlace-mode=progressive,colorimetry=bt709
audiocaps=audio/x-raw,format=S16LE,channels=8,layout=interleaved,rate=48000

sources=cam1,cam2,cam3,slides
livesources=slides

[source.cam1]
kind=decklink
devicenumber=1
video_connection=SDI
video_mode=1080p25
volume=1.0
audio.original = 0+1

[source.cam2]
kind=test
volume=1.0
audio.test = 0+1

[source.cam3]
kind=test
volume=0.0

[source.slides]
kind=decklink
devicenumber=2
video_connection=SDI
video_mode=1080p25
volume=0.0

Will try to find out why, if i can't figure it out i might have to revert this.

@Kunsi
Copy link
Contributor

Kunsi commented Nov 23, 2024

fixed in 1563f9e - no clue why noone spotted this earlier

@justjanne justjanne deleted the fix-audio-only-sources branch December 21, 2024 18:00
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.

2 participants