Re: [webrtc-stats] Add PSNR (Y/U/V) for outbound-rtp (#794)

This seems like a nice feature that could have a few uses. I do wonder if it could be a separate API instead of part of the outbound RTP stats. My initial concerns are calculating this data regardless if the application is even interested in the data and no specification or recommendations on frequency of measurements.

Some pros and cons that come to mind if this were implemented as a separate API instead.
Pros:
- This would allow for those interested in the information to request a PSNR directly. 
- Eliminates the need to constantly be collecting PSNR measurements that may not be used.
- It would allow for applications to have some control over the frequency of PSNR measurements.

Cons:
- Applications would not be getting PSNR sum for free and need to calculate and store this on their own.

If this should remain in the stats could we consider adding some sort of getStats object to enable logging for this kind of data?

-- 
GitHub Notification of comment by Drekabi
Please view or discuss this issue at https://github.com/w3c/webrtc-stats/pull/794#issuecomment-2686570972 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Thursday, 27 February 2025 01:22:44 UTC