Skip to content

Add frame counters#125

Merged
guidou merged 9 commits into
mainfrom
guidou/mstp-counters
Apr 16, 2026
Merged

Add frame counters#125
guidou merged 9 commits into
mainfrom
guidou/mstp-counters

Conversation

@guidou

@guidou guidou commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

@guidou guidou requested review from jan-ivar and youennf April 8, 2026 15:51
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
guidou and others added 2 commits April 9, 2026 16:42
Co-authored-by: Jan-Ivar Bruaroey <jan-ivar@users.noreply.github.com>
Comment thread index.bs Outdated
Comment thread index.bs Outdated
guidou and others added 2 commits April 16, 2026 16:14
Co-authored-by: Jan-Ivar Bruaroey <jan-ivar@users.noreply.github.com>
Co-authored-by: youennf <youennf@users.noreply.github.com>
Comment thread index.bs
Comment thread index.bs
@youennf

youennf commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

The application may detect that frames have been dropped by noticing that there is a gap in the timestamps of the frames.

We should probably update that note. Maybe:
The application may detect that frames have been dropped by noticing that there is a gap in the timestamps of the frames or via numDiscardedFrames

Comment thread index.bs
specific situations. In this case, `[[numDiscardedFrames]]` MUST be incremented
accordingly. In all cases, frames that are not dropped must be made available
to the {{ReadableStream}} in the order in which they arrive to the
{{MediaStreamTrackProcessor}}.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We could remove or reduce this sentence given this is already stated. I filed #126.

@guidou guidou merged commit cd21bb6 into main Apr 16, 2026
2 checks passed
github-actions Bot added a commit that referenced this pull request Apr 16, 2026
SHA: cd21bb6
Reason: push, by guidou

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants