KyroFrCode / moonlight-chrome-tizen

GameStream client for Tizen OS (Samsung Smart TV's)
GNU General Public License v3.0
90 stars 18 forks source link

Add 120 fps option #9

Open glink00 opened 1 year ago

glink00 commented 1 year ago

Attempt to add 120 FPS

Changes proposed in this pull request:

I guess this would partially take care of https://github.com/KyroFrCode/moonlight-chrome-tizen/issues/2

Sunshine logs:

image

@moonlight-stream

Velka-DEV commented 1 year ago

Tested personally, this break the application when we launch a game at STAGE_VIDEO_STREAM_START stage. (Video establishment screen)

glink00 commented 1 year ago

What model did you test on? On my S95B the app works like normal. Even though on the sunshine logs I do see 120 fps requested, there is no performance overlay for me to validate if it is actually 120 on stream.

Velka-DEV commented 1 year ago

What model did you test on? On my S95B the app works like normal. Even though on the sunshine logs I do see 120 fps requested, there is no performance overlay for me to validate if it is actually 120 on stream.

My model is SAMSUNG - 55Q70A, and i run the PC with a RTX 3070 with latest driver

EDIT: A lot of TV have 100hz native with 120hz on HDMI. I don't really know if the max HZ should count on the native or HDMI like for moonlight protocol

KyroFrCode commented 1 year ago

This implementation breaks the app or the app works without a hitch, I'll try to add an overlay option when I can to see all the logs and settings of the stream directly on the TV. I'll try on my hand this implementation.

AndrejsVas commented 11 months ago

built your code and chose 120fps in the client. Stream starts, sunshine says that 120fps is requested, but the actual result is definitely not 120fps. I have no overlay, but side to side comparison leaves no room for doubt.