Skip to content

drifttracer: add cumulativeOverdrift and expose via stats#2481

Closed
oviano wants to merge 1 commit into
Haivision:masterfrom
oviano:drifttracer-cumulative-overdrift
Closed

drifttracer: add cumulativeOverdrift and expose via stats#2481
oviano wants to merge 1 commit into
Haivision:masterfrom
oviano:drifttracer-cumulative-overdrift

Conversation

@oviano
Copy link
Copy Markdown
Contributor

@oviano oviano commented Oct 6, 2022

The cumulative overdraft represents the clock drift since the start of the stream. Exposing this information via the stats system allowed my video player to adjust its playback speed to accommodate the clock drift and keep everything sweet.

@maxsharabayko maxsharabayko added Type: Enhancement Indicates new feature requests [API] Area: Changes in SRT library API labels Oct 7, 2022
@maxsharabayko maxsharabayko modified the milestones: v1.6.0, Major Oct 7, 2022
@maxsharabayko
Copy link
Copy Markdown
Collaborator

Touches SRT API. Might be something to consider within #1215.

@oviano oviano closed this by deleting the head repository Jan 12, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 12, 2026

Codecov Report

❌ Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.97%. Comparing base (66179ab) to head (921b8b5).
⚠️ Report is 350 commits behind head on master.

Files with missing lines Patch % Lines
srtcore/core.cpp 33.33% 2 Missing ⚠️
srtcore/utilities.h 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2481      +/-   ##
==========================================
- Coverage   66.12%   65.97%   -0.15%     
==========================================
  Files          96       96              
  Lines       19602    19609       +7     
==========================================
- Hits        12961    12937      -24     
- Misses       6641     6672      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

[API] Area: Changes in SRT library API Type: Enhancement Indicates new feature requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants