ALVR icon indicating copy to clipboard operation
ALVR copied to clipboard

feature request: Configurable color space

Open nowak-pl opened this issue 1 year ago • 3 comments

Currently ALVR sets color space to DCI-P3 in client. https://github.com/alvr-org/ALVR/blob/4d0d0ce59ac6ffe34b4e19f5fc8bfe25f6ab8202/alvr/client_openxr/src/lib.rs#L245

It would be nice to have selectable color space in streamer because content authored in Rec. 709 or sRGB doesn't look the best when expanded to P3 without proper conversion and then clipped to HMD LCD panel native color space.

nowak-pl avatar Sep 25 '24 22:09 nowak-pl

We are limited by the codec standards. It is best not to needlessly change the color space before encoding because that will exacerbate color aliasing (banding).

Or I may be misinterpreting your question. We could add an option on the streamer to instruct the client to use a specific color space.

zmerp avatar Sep 25 '24 23:09 zmerp

I'm only talking about sending to client which color space should it use in set_color_space call, no conversions on either end.

nowak-pl avatar Sep 25 '24 23:09 nowak-pl

Yeah it can be done. There is a openxr extension to do that. I'll accept PR to implement it.

zmerp avatar Sep 26 '24 00:09 zmerp