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

Temporarily remove L1T3 SVC mode test #41876

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Djuffin
Copy link
Contributor

@Djuffin Djuffin commented Sep 8, 2023

Also use latencyMode = realtime

Also use latencyMode = realtime
@wpt-pr-bot
Copy link
Collaborator

There are no reviewers for this pull request. Please reach out on the chat room to get help with this. Thank you!

@Djuffin Djuffin requested a review from youennf September 8, 2023 07:34
@Djuffin
Copy link
Contributor Author

Djuffin commented Sep 8, 2023

Addressing web-platform-tests/interop#375

@youennf
Copy link
Contributor

youennf commented Sep 8, 2023

In realtime mode, it is allowed to drop frames, either if the encoder cannot keep up with the frame rate or to cope with the bit rate. I am not sure how big this is an issue in practice but the test could be rewritten to take this into account.
Also, keeping L1T3 coverage might still be good in some Chrome setups, just not for Interop 2023.

@padenot
Copy link
Contributor

padenot commented Aug 29, 2024

We can likely just close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants