-
Notifications
You must be signed in to change notification settings - Fork 28
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
More timestamp improvements #102
base: master
Are you sure you want to change the base?
Conversation
…icRefCell instead of Mutex
Audio/video are in practice not always from the same clock and can have different behaviours with regards to clock rate and jitter. Handling them separately generally gives better results for the timestamps output by the source element.
- NDI HX Camera Android in the past used 1ns instead of 100ns as unit for timecodes/timestamps. - NDI HX Camera iOS uses 0 for all timecodes and the same non-zero value for all audio timestamps Detect such situations and try to compensate for them. Also add a new "auto" timestamping mode that prefers to use timecodes and otherwise falls back to timestamps or receive times. Fixes teltek#79
This pull request also fixes a problem that I noticed, where by using the source and sink pipelines specified in the |
That's probably the 3rd commit |
I've pulled this down and tested it and it helps with some problematic NDI sources using gstreamer, specifically the NDI camera app on an iphone seems to work well using the |
When using the Advanced SDK it is possible to output compressed formats too.
This is merged here now: https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/tree/main/net/ndi |
See individual commits.
CC @samuvlad @pnieto