webrtc-issue-detector icon indicating copy to clipboard operation
webrtc-issue-detector copied to clipboard

How to distinguish MOS in group call?

Open IlgamGabdullin opened this issue 1 year ago • 3 comments

Hi! How can I distinguish inbound MOS if I have several participants in videocall?

I want to connect partner id's and inbound mos value

IlgamGabdullin avatar May 06 '24 09:05 IlgamGabdullin

I'd want to make a pr with the needed changes which lets us to distinguish several inbound stats by some id.

As I saw in the code, CompositeStatsParser.addPeerConnection has 'id' in params (interface AddConnectionPayload), but it never gets back to the client.

The possible solution I see is to change params of method handleNewPeerConnection(pc) to handleNewPeerConnection(pc, id?), then pass the id to addPeerConnection({pc, id}), and emit the id back to the client in callbacks like onIssues and onNetworkScoresUpdated

If you have any concerns on why the solution above might not work, lets discuss

IlgamGabdullin avatar May 08 '24 06:05 IlgamGabdullin

Hi @IlgamGabdullin, thank your for reaching out. If I got your question right, you can make the match by ssrc that is returned with issues. Your suggested changes look useful, but it's better if you could create a PR so that our community could look through the suggested changes. Please, try not to make breaking changes in your proposal PR

evgmel avatar May 08 '24 07:05 evgmel

make the match by ssrc that is returned with issues.

Thank you for your answer! Seems like issues could be matched as you said, but mos could not (does not have ssrc field in NetworkScores).

I will come back with pr

IlgamGabdullin avatar May 08 '24 08:05 IlgamGabdullin

@IlgamGabdullin thank you for your contribution! Look forward to further cooperation

desher avatar May 16 '24 05:05 desher