Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Max FPS not honouring the URL parameter with 5.1 and not working at all with 5.0 #100

Closed
dan-tw opened this issue Feb 17, 2023 · 7 comments · Fixed by #134
Closed

Max FPS not honouring the URL parameter with 5.1 and not working at all with 5.0 #100

dan-tw opened this issue Feb 17, 2023 · 7 comments · Fixed by #134

Comments

@dan-tw
Copy link
Collaborator

dan-tw commented Feb 17, 2023

On the settings panel under WebRTC there is a Max FPS setting. If value in the panel is changed, it appends a parameter to the URL e.g. http://localhost/?WebRTCFPS=30

Using UE 5.1

  • Visiting the frontend URL with the ?WebRTCFPS=30 parameter does not update the Max FPS in the Information panel.
  • Changing the Max FPS in the WebRTC section on the Settings panel does update the MaxFPS in the Information panel.

Using UE 5.0, Max FPS just does not seem to work at all

@dan-tw dan-tw changed the title Max FPS not honouring the URL parameter Max FPS not honouring the URL parameter with 5.1 and not working at all with 5.0 Feb 17, 2023
@Belchy06
Copy link
Collaborator

I've got an idea of what's happening with this one.

I think what's happening is we're setting the FPS to what ever is in the URL. Updating this does nothing as when first run we currently don't have a stream.

User starts a stream, and will receive the initial settings object which updates all of WebRTC and encoder settings.

The callbacks on these settings are set to send their new value to UE so we end up just setting them back to whatever the application send us in the first place.

@lukehb
Copy link
Contributor

lukehb commented Feb 27, 2023

@MWillWallT Is this fixed now, I think it is?

@MWillWallT
Copy link
Contributor

@lukehb I've tested this, unfortunately the issues are still present.
5.1 doesn't update the FPS in the info panel with the URL /?WebRTCFPS=30. As reported though changing it via the control panel does work. This was tested with the 5.1 Infra and latest 5.1.

I'm currently in the process of pulling 5.0 to test that.

@Belchy06
Copy link
Collaborator

It's a frontend specific issue. Will still happen with 5.0

@lukehb
Copy link
Contributor

lukehb commented Feb 27, 2023

Is it fixed in the UE 5.2 branch of the infra. If so, we can backport.

@Belchy06
Copy link
Collaborator

This was tested using latest frontend version. Pretty sure we didn't support setting WebRTC settings in the URL with the old JS monolith, which is why it's popped up now.

@lukehb
Copy link
Contributor

lukehb commented Mar 1, 2023

Fixed in 1695df2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
4 participants