-
Notifications
You must be signed in to change notification settings - Fork 18
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
Validate IngressInfo and populate missing fields with defaults #90
Conversation
pkg/media/presets.go
Outdated
"github.com/livekit/ingress/pkg/errors" | ||
"github.com/livekit/protocol/livekit" | ||
) | ||
|
||
const ( | ||
// reference parameters used to compute bitrate | ||
refBitrate = 3_000_000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you mentioned bumping this up a bit. wdyt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The most pressing is to increase the bitrate of the medium and low layers I think. We do this here by taking the power to 0.75 of the pixel area ratio. This was chosen to double the bitrate of the low layer (1/4 width and 1/4 height), compared to scaling the bitrate linearly.
We should probably reassess the 3Mbps value for 1080p content as well, but I'd want more time and more content to test to pick a value.
This is the v1.0.0 release of the ingress service. This release stabilizes the API and adds support for the WHIP protocol ## Changelog ### Added - WHIP support (#97) - Expose heath and availability on the WHIP HTTP server (#109) - Add support for optimally bypassing transcoding when ingesting WHIP (#111) - Set the source track information in the ingress state (#121) ### Fixed - Validate IngressInfo and populate missing fields with defaults (#90) - Fix deadlock in appsrc (#116) - Update synchronizer and use Jitter Buffer (#118)
Also run until tests on pull requests.