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

Streaming output support #41

Open
rafael2k opened this issue Jan 24, 2022 · 2 comments
Open

Streaming output support #41

rafael2k opened this issue Jan 24, 2022 · 2 comments

Comments

@rafael2k
Copy link

Hi there,
I'm implementing streaming output support to talked. I implemented support, for now just for "audio_only = True" use case (and tested with icecast output):
https://github.com/ColmenaDev/talked
Please advice on what should go to json parameters and what should go to config file. Also, ffmpeg in docker is too old, and tee plugin is not working with onfail (so for now I'm using two encoding pipelines in ffmpeg available in ubuntu 20.04). It would be nice to update dockerfile to Debian 11 or Ubuntu 22.04.

@mwalbeck
Copy link
Contributor

Hi there, I'm implementing streaming output support to talked. I implemented support, for now just for "audio_only = True" use case (and tested with icecast output)

Very cool!

Please advice on what should go to json parameters and what should go to config file.

I would say having both of the new config options in the config file makes sense, so one can set a default. For JSON parameters I would say having the enable_streaming option present makes sense, so one can enable or disable streaming through the Talked client for individual talk rooms. Whether it makes sense to include icecast_url depends on whether you see a use case for setting the icecast_url through the Talked client, that overrides the default set in the config file.

Also, ffmpeg in docker is too old, and tee plugin is not working with onfail (so for now I'm using two encoding pipelines in ffmpeg available in ubuntu 20.04). It would be nice to update dockerfile to Debian 11 or Ubuntu 22.04.

The dockerfile is currently just used for development, so you're welcome to update it. Updating to a newer ubuntu will probably be easiest, as Ubuntu includes the geckodriver, that is needed for selenium, in their repos and Debian doesn't.

Though when I create a docker container for production use it'll probably be Debian based as that is what I normally use for all my containers.

@rafael2k
Copy link
Author

btw, I'm testing with icecast://, but streaming like could be udp://, hsl://, or whatever other streaming protocol ffmpeg supports could also be used.

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

No branches or pull requests

2 participants