Skip to content

network: Check max recommended bitrate from service#167

Merged
RytoEX merged 1 commit intoobsproject:masterfrom
prgmitchell:checkServiceRecs
Jul 2, 2025
Merged

network: Check max recommended bitrate from service#167
RytoEX merged 1 commit intoobsproject:masterfrom
prgmitchell:checkServiceRecs

Conversation

@prgmitchell
Copy link
Member

@prgmitchell prgmitchell commented Jun 24, 2024

Description

This adds a warning if the maximum recommended video bitrate from the selected service has been exceeded.

image

Requires obsproject/obs-studio#10917 to be merged first.

Motivation and Context

Added logging to OBS to display max bitrate when the user elects to ignore the service recommendations, this brings that to the analyzer as well.

How Has This Been Tested?

Tested with this log https://obsproject.com/logs/ZuadI85OgoP0Yjjj

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@prgmitchell prgmitchell force-pushed the checkServiceRecs branch 2 times, most recently from 3202bfa to 5594051 Compare June 25, 2024 19:53
@RytoEX RytoEX marked this pull request as draft November 21, 2024 21:59
@RytoEX
Copy link
Member

RytoEX commented Jun 28, 2025

Looks like #11733 (the replacement for #10917) was merged, so this can be undrafted. This will require a rebase.

@RytoEX RytoEX marked this pull request as ready for review June 28, 2025 23:35
@prgmitchell
Copy link
Member Author

@RytoEX updated, seems to be working fine

image

The build is failing though, looks like the test it runs is pointing at a log file that no longer exists.

@Warchamp7
Copy link
Member

@RytoEX updated, seems to be working fine
The build is failing though, looks like the test it runs is pointing at a log file that no longer exists.

#183 is to fix that

@RytoEX RytoEX requested review from Fenrirthviti and RytoEX July 2, 2025 19:20
@RytoEX RytoEX self-assigned this Jul 2, 2025
@Fenrirthviti
Copy link
Member

Just for my reference, this warning only shows if the ignore recommendations option is enabled, and it's actually sending over the service's configured limit from services.json?

@prgmitchell
Copy link
Member Author

Just for my reference, this warning only shows if the ignore recommendations option is enabled, and it's actually sending over the service's configured limit from services.json?

Correct

This adds a warning if the maximum recommended video bitrate from the selected service has been exceeded.
@RytoEX RytoEX force-pushed the checkServiceRecs branch from d26f983 to 9d092f3 Compare July 2, 2025 20:51
@RytoEX RytoEX merged commit df1f861 into obsproject:master Jul 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants