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.

Sign up for GitHub

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

Jump to bottom

Add Player stats and improve Telemery #11

Open
wants to merge 20 commits into
base: dev
Choose a base branch
from
Open

Add Player stats and improve Telemery #11

wants to merge 20 commits into from

Conversation

thomasjammet
Copy link
Collaborator

No description provided.

@thomasjammet
chore(HTTPConnector): handle correctly a signaling fetch error
1d93932 
WHEP connection on offline stream was trying to call setRemoteDescription
@thomasjammet
fix(StreamerStats): fix video metrics
ee5cdd2
@thomasjammet
style(StreamerStats): fix a comment
fae126b
@thomasjammet
feat(PlayerStats): add playerstats class to implement basic player me…
310ad2d 
…trics
@thomasjammet
docs(Connectors): small typo and comments changes
6592804
@thomasjammet
chore(PlayerStats): add incomingBitrate
60837b5 
This is in addition of the availableIncomingBitrate which is an estimation of the bandwidth capacity
@thomasjammet
chore(Player): make connectioninfos cacheduration configurable
b201c24 
This parameter must be change with care as it can affect CPU consumption if too low.
@thomasjammet
feat(Telemetry): change telemetry report() behavior
4fb8ce1 
Now calling report() with the same statistics class and a frequency will update the previous frequency
@thomasjammet
chore(Streamer): make connectioninfos cacheduration configurable
c73ebb3
@thomasjammet
chore(examples): add player-stats.html example
c9c3534 
This example shows how to use the statistics and telemetry classes.
@thomasjammet
docs(README): document the telemetry + add example
bc21bab
@thomasjammet
chore(example): remove customstats timestamp property
d4f7352
@thomasjammet
Use web-utils (CeeblueTV#10)
3d88029 
* build(web-utils): use @ceeblue/web-utils and remove src/utils

* build(node-resolve): add rollup/plugin-node-resolve dependency to include web-utils in the bundle

* chore(version): add version constant in bundle to replace util.version

* build: update web-utils

* feat(index.ts): export utils, the web-utils dependency

BREAKING CHANGE: replace utils with web-utils dependency
@thomasjammet
Merge branch 'dev' into player-stats
e3fd7f3
@thomasjammet
chore(PlayerStats): update playerstats to use web-utils
c182d8b
@thomasjammet
chore(player-stats.js): improvement to show the timestamps correctly
5de0989
@thomasjammet
chore(CustomStats.js): Correction of custom metrics NaN values
98c7f88
@thomasjammet
chore(player-stats.html): UI improvement for responsiveness
5bfb81f
@thomasjammet
chore(examples): update webfonts
f967eb9 
There was a warning on firefox
@thomasjammet
Merge branch 'dev' into player-stats
4cd3175
@MathieuPOUX
Copy link
Collaborator

MathieuPOUX commented Apr 25, 2024 •
edited
Loading

  1. IStats + Telemetry can move to web-utils
  2. Combine player-stats.js and player-stats.html into a unique html example file like the other example (player.html & streamer.html)
  3. Remove CustomStats.ts => player-stats.js must illustrate how use PlayerStats.ts
  4. Move the need of customer stats (CustomStats.ts) to a different project (https://github.com/CeeblueTV/examples? or branch)

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

@jobl jobl Awaiting requested review from jobl

@MathieuPOUX MathieuPOUX Awaiting requested review from MathieuPOUX

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@thomasjammet @MathieuPOUX