Skip to content

Add optional AV1 codec negotiation (#22) - #43

Merged
hugoalves-dev merged 30 commits into
mainfrom
codex/issue-22-av1
Sep 29, 2026
Merged

hugoalves-dev merged 30 commits into
mainfrom
codex/issue-22-av1

Conversation

@hugoalves-dev

@hugoalves-dev hugoalves-dev commented Sep 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Add opt-in hardware AV1 probing and Media Foundation codecs while retaining H.264 fallback.
  • Negotiate a shared codec from publisher/viewer capabilities and required AV1 profile/level.
  • Add AV1 SDP/RTP handling, including sequence continuity checks across temporal units; gaps drop the affected unit and trigger the existing recovery keyframe request.
  • Cap advertised AV1 level to the 640x360@30 configuration actually exercised by the capability probe.
  • Keep audio flowing and video queues gated on a fresh keyframe through same-peer H.264 fallback, including runtime AV1 encoder failures.
  • Surface codec decisions/fallback reasons and add a reproducible opt-in encoder benchmark.

Validation

  • RTC Release test project: 144 passed.
  • Windows media Release test project: 153 passed, 2 skipped.
  • AV1 RTP assembler tests: 15/15; AV1 capability probe tests: 6/6.
  • The Windows CI failures from the prior run were traced to RTC fakes missing same-connection H.264 offer support and an overstrict capture-burst send-count assertion. Both are corrected.

The benchmark compares AV1 and H.264 at 640x360 and 30 fps. It is synthetic and encoder-only; decoder performance, capture cadence, RTP/network behavior, end-to-end latency, dropped frames, and visual quality are unavailable and are not inferred.

@hugoalves-dev
hugoalves-dev marked this pull request as ready for review September 29, 2026 09:19

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 10893755d1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@hugoalves-dev

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fb6b221f21

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@hugoalves-dev
hugoalves-dev merged commit 37e6383 into main Sep 29, 2026
4 checks passed
@hugoalves-dev
hugoalves-dev deleted the codex/issue-22-av1 branch September 29, 2026 15:09
@hugoalves-dev hugoalves-dev linked an issue Sep 29, 2026 that may be closed by this pull request
12 tasks
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.

Add optional AV1 codec negotiation when both peers support it

1 participant