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 layer dimension calculation logic #143

Merged
merged 14 commits into from
Aug 14, 2023
Merged

Conversation

biglittlebigben
Copy link
Contributor

Change layer dimension calculation logic to respect source spec ratio, and never upscale original frame. Also, prune layers that are too large for the original frame.

@biglittlebigben biglittlebigben merged commit 0055462 into main Aug 14, 2023
2 checks passed
@biglittlebigben biglittlebigben deleted the benjamin/resolution_fixes branch August 14, 2023 22:38
@biglittlebigben biglittlebigben mentioned this pull request Aug 16, 2023
biglittlebigben added a commit that referenced this pull request Aug 17, 2023
This is the v1.1.0 release of the ingress service. This release adds support for pulling media from URLs and fixes source resolution handling
  
 ## Changelog
    
 ### Added
- Log redacted ingress info and set stereo and disableDTX publication options (#130)
- Add Debug handler with pprof support (#131)
- Report average bitrate in the ingress state (#133)
- Support for GST pipeline debug dot files on the debug handler (#136)
- Support for pulling URLs (#138)
    
 ### Fixed
- Do not report instance as available while draining (#124)
-  Handle better the case where 2 publishers with the same ingress ID connect to the same ingress instance (#125)
- Check stream key in WHIP DELETE call (#126)
- Make sure handler processes use the same node id as the service (#129)
- Update the WebRTC settings engine to match the sessions parameters (#139)
- Adopt jitter.PopSamples in the WHIP handler (#141)
- Only accept specific file types in url pull (#142)
- Fix layer dimension calculation logic (#143)
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