Skip to content

Support RGB colorspaces and explicit tone mapping #126

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
ns6089 wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Support RGB colorspaces and explicit tone mapping #126

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3941843
Support RGB colorspaces and explicit tone mapping
ns6089 Feb 18, 2026
a840059
fixup! Support RGB colorspaces and explicit tone mapping
ns6089 Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 16 additions & 3 deletions src/Limelight.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,14 @@ extern "C" {

// Values for the 'colorSpace' field below.
// Rec. 2020 is not supported with H.264 video streams on GFE hosts.
#define COLORSPACE_REC_601 0
#define COLORSPACE_REC_709 1
#define COLORSPACE_REC_2020 2
// To use any of the RGB colorspaces, the following conditions must be satisfied:
// 1. Server advertised SCM_MASK_RGB SCM flag in addition to individual SCM formats from SCM_MASK_YUV444
// 2. Client included these 4:4:4 formats in 'supportedVideoFormats' field of STREAM_CONFIGURATION
#define COLORSPACE_REC_601 0
#define COLORSPACE_REC_709 1
#define COLORSPACE_REC_2020 2
#define COLORSPACE_RGB_SRGB 3
#define COLORSPACE_RGB_P3_D65 4

// Values for the 'colorRange' field below
#define COLOR_RANGE_LIMITED 0
Expand Down Expand Up @@ -100,6 +105,12 @@ typedef struct _STREAM_CONFIGURATION {
// in /launch and /resume requests.
char remoteInputAesKey[16];
char remoteInputAesIv[16];

// If specified, request the encoder to tone map HDR content.
// If not set and selected format is 10-bit, HDR content will be encoded as is.
// The reason behind this field is the ambiguity of 'supportedVideoFormats',
// as it's impossible to request 10-bit tone mapped content through that field alone.
int toneMapHDR;
Copy link
Contributor Author

@ns6089 ns6089 Feb 18, 2026

Choose a reason for hiding this comment

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

Not sure which one is better, pushing like this or pulling through renderer capabilities.

} STREAM_CONFIGURATION, *PSTREAM_CONFIGURATION;

// Use this function to zero the stream configuration when allocated on the stack or heap
Expand Down Expand Up @@ -520,6 +531,8 @@ void LiInitializeConnectionCallbacks(PCONNECTION_LISTENER_CALLBACKS clCallbacks)
#define SCM_MASK_AV1 (SCM_AV1_MAIN8 | SCM_AV1_MAIN10 | SCM_AV1_HIGH8_444 | SCM_AV1_HIGH10_444)
#define SCM_MASK_10BIT (SCM_HEVC_MAIN10 | SCM_HEVC_REXT10_444 | SCM_AV1_MAIN10 | SCM_AV1_HIGH10_444)
#define SCM_MASK_YUV444 (SCM_H264_HIGH8_444 | SCM_HEVC_REXT8_444 | SCM_HEVC_REXT10_444 | SCM_AV1_HIGH8_444 | SCM_AV1_HIGH10_444)
#define SCM_MASK_RGB 0x80000000 // Indicates that YUV444 profiles also support RGB payload
#define SCM_MASK_RECOMB 0x40000000 // Indicates that YUV420 profiles also support recombined YUV444 payload

typedef struct _SERVER_INFORMATION {
// Server host name or IP address in text form
Expand Down