Skip to content

Participant joined_at_ms to get higher resolution of join time. - #949

Merged
boks1971 merged 2 commits into
mainfrom
raja_joined_at_ms
Jan 22, 2025
Merged

Participant joined_at_ms to get higher resolution of join time.#949
boks1971 merged 2 commits into
mainfrom
raja_joined_at_ms

Conversation

@boks1971

@boks1971 boks1971 commented Jan 22, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@changeset-bot

changeset-bot Bot commented Jan 22, 2025 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1576bb4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR 
Some errors occurred when validating the changesets config:
The package or glob expression "github.com/livekit/protocol" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

@boks1971
boks1971 requested a review from a team January 22, 2025 08:14
@boks1971
boks1971 merged commit 18fcdd8 into main Jan 22, 2025
@boks1971
boks1971 deleted the raja_joined_at_ms branch January 22, 2025 08:18
@bcherry

bcherry commented Jan 22, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

would it make sense to add a Room.creation_time_ms while we're at it? the client SDKs consume these in the same way and I also imagine there are developers doing math on room creation time vs participant join time so they should have the same resolution...

@boks1971

boks1971 commented Jan 23, 2025

Copy link
Copy Markdown
Contributor Author

Will look at adding that @bcherry . One note though. room creation time and participant join time could be using different clocks in cloud as they could be on different nodes. But, it should be within the clock skew of servers which is probably okay for the kind of comparisons you have mentioned.

boks1971 added a commit that referenced this pull request Jan 23, 2025
@boks1971
Room creation time with ms resolution.
fccb6aa 
Follow up to this PR - #949 and
this comment - #949 (comment)
@boks1971 boks1971 mentioned this pull request Jan 23, 2025
Merged
@boks1971

boks1971 commented Jan 23, 2025

Copy link
Copy Markdown
Contributor Author

Protocol change PR - #950

boks1971 added a commit that referenced this pull request Jan 23, 2025
@boks1971 @github-actions
Room creation time with ms resolution. (#950)
6ec1903 
* Room creation time with ms resolution.

Follow up to this PR - #949 and
this comment - #949 (comment)

* generated protobuf

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
@russellmartin-livekit russellmartin-livekit mentioned this pull request Jul 10, 2026
Merged
2 tasks
russellmartin-livekit added a commit that referenced this pull request Jul 29, 2026
@russellmartin-livekit
generated: add realtime reporter methods to gatewayobs
46aa294 
Regenerated from cloud-observability schema/gateway.yaml realtime (speech-to-speech)
billing columns (#502). Adds ReportRealtime{DurationSeconds, Input{Text,Audio,Image,
Video}Tokens, Output{Text,Audio}Tokens, Cached{Text,Audio}Tokens} to the gateway
model reporter, so agent-gateway can emit the realtime_* metrics that populate the
observability rollup consumed by analytics-recorder (#949) and backend-common (#939).
russellmartin-livekit added a commit that referenced this pull request Jul 29, 2026
@russellmartin-livekit @github-actions
generated: add realtime reporter methods to gatewayobs (#1663)
b7ca5c9 
* generated: add realtime reporter methods to gatewayobs

Regenerated from cloud-observability schema/gateway.yaml realtime (speech-to-speech)
billing columns (#502). Adds ReportRealtime{DurationSeconds, Input{Text,Audio,Image,
Video}Tokens, Output{Text,Audio}Tokens, Cached{Text,Audio}Tokens} to the gateway
model reporter, so agent-gateway can emit the realtime_* metrics that populate the
observability rollup consumed by analytics-recorder (#949) and backend-common (#939).

* generated protobuf

---------

Co-authored-by: github-actions <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

Reviewers

@cnderrauber cnderrauber cnderrauber approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@boks1971 @bcherry @cnderrauber