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

[Audio Stats] Add current latency #119

Closed
henbos opened this issue Oct 19, 2023 · 2 comments · Fixed by #120 or #124
Closed

[Audio Stats] Add current latency #119

henbos opened this issue Oct 19, 2023 · 2 comments · Fixed by #120 or #124
Assignees

Comments

@henbos
Copy link
Contributor

henbos commented Oct 19, 2023

While discussing #117, it was pointed out that in addition to "avg latency" computed from two track.stats calls, for A/V sync it would be super useful to know what the latest latency was. We also removed total latency from that PR in order to separate discussions and make progress.

In this issue we'll address current latency. I'll create a separate issue to address average latency.

@henbos
Copy link
Contributor Author

henbos commented Oct 28, 2023

What the heck, did my PRs get so engagled that GitHub merged both of them? I'll create a new one, latency has NOT been fixed.

@henbos henbos reopened this Oct 28, 2023
@henbos henbos changed the title [Audio Stats] Current latency is also useful [Audio Stats] Add current and total latency Oct 28, 2023
@henbos
Copy link
Contributor Author

henbos commented Oct 28, 2023

The new audio latency PR is #124, it has not merged yet.

@henbos henbos self-assigned this Nov 8, 2023
@henbos henbos changed the title [Audio Stats] Add current and total latency [Audio Stats] Add current latency Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment