Skip to content
This repository was archived by the owner on Feb 20, 2026. It is now read-only.

VIDCS-4092: Simple public sample for sender-side stats for subscribers in Android - #554

Merged
goncalocostamendes merged 5 commits into
mainfrom
VIDCS-4092
Dec 19, 2025
Merged

goncalocostamendes merged 5 commits into
mainfrom
VIDCS-4092

Conversation

@goncalocostamendes

Copy link
Copy Markdown
Contributor

Simple public sample for sender-side stats for subscribers in Android

https://jira.vonage.com/browse/VIDCS-4092

@goncalocostamendes goncalocostamendes changed the title VIDCS-4092: Add documentaiton on README VIDCS-4092: Simple public sample for sender-side stats for subscribers in Android Oct 1, 2025

@juliobecerragomez juliobecerragomez 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.

LGTM


## Code sample - Subscriber Stats

The user can retrieve video stats, conatining information such as ```videoPacketsLost``` and ```videoPacketsReceived``` by implementing listener SubscriberKit.VideoStatsListener.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Typo. containing

publisher = new Publisher.Builder(MainActivity.this).senderStatsTrack(true).build();
session.publish(publisher);


Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nit. too many empty lines?

}
};

SubscriberKit.VideoStatsListener videoStatsListener = new SubscriberKit.VideoStatsListener() {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I understand it only shows subscriber side stats because we want to focus on sender-side stats. Maybe we can extend with all the new (mainly publisher) ones after the 2.33 release.

@goncalocostamendes
goncalocostamendes merged commit 816bac5 into main Dec 19, 2025
40 of 41 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants