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

Add explainer for encoded frame creation #164

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

alvestrand
Copy link
Contributor

This helps set the background for API change proposals.

This helps set the background for API change proposals.
Copy link
Collaborator

@fippo fippo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍰

create-encoded-explainer.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@youennf youennf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for writing this explainer, this is helpful to drive the discussion forward.
At this stage of the discussion, I think we should dive further into use cases to extract technical requirements we could put on existing APIs (or identify missing new APIs).
This would also allow adding these technical requirements in this document and link them to NV use cases.

create-encoded-explainer.md Outdated Show resolved Hide resolved
create-encoded-explainer.md Outdated Show resolved Hide resolved
create-encoded-explainer.md Outdated Show resolved Hide resolved

In some use cases, we need to move encoded data beyond the confines of a single PeerConnection.

The use cases include:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Below use cases are closer to technical solutions than actual use cases.
It would be helpful to relate these technical solutions to NV document use cases.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have filed w3c/webrtc-nv-use-cases#81 and w3c/webrtc-nv-use-cases#82 in order to provide a discussion point for those use cases, before spending the time to write actual text for the use case.

w3c/webrtc-nv-use-cases#79 has been merged, so that's part of the NV-use-cases now.

create-encoded-explainer.md Show resolved Hide resolved
@dontcallmedom-bot
Copy link

This issue was mentioned in WEBRTCWG-2023-01-17 (Page 30)

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.

4 participants