-
-
Notifications
You must be signed in to change notification settings - Fork 8
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.
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
Performance regression in Firefox version 123 & 124 #224
Comments
Hi @Askejm, let's try to figure out what caused the sudden drop in performance. I'm on a Nvidia 2070 Super, so I might be able to reproduce your issue. Some follow up questions:
|
Thanks for the quick response @WesselKroos
|
It looks like you've found a Firefox version 123 bug. I hit the same framerate ~12 fps limit for a 4K video in Firefox. But after downgrading to Firefox 122 I was able to get 60fps. I don't have a workaround for you at this moment. But I'll report this performance regression to the Firefox developers. Let's hope we can find a workaround for Firefox 123 in the mean time. Running Firefox versions 122 and 123 side-by-side: |
Oh thats odd. Well ill downgrade then, thanks! |
The bug has been reported to the Firefox developers: |
@Askejm I think I found a workaround. Could you verify that executing this javascript code on the /watch page fixes the bug in Firefox 123? ambientlight.enableMozillaBugReadPixelsWorkaround = false Because it looks like they have fixed bug 1719154 while introducing bug 1883225 |
@WesselKroos Looks like that did it. |
Then I'll turn off that workaround from Firefox 123. I'll also make the workaround available via an advanced setting. Just in case Firefox decides to revert their change in 124. |
Did you push this update? I still get bad fps without running the javascript. Version 2.38.1 @WesselKroos |
@Askejm Not yet, it will be released in 2.38.2. |
Those are some large consistent spikes, more than 2x the normal duration. My graph seems to be a lot smoother: Even when Firefox's rendering process is a half display frame desynced from the video frame (this alternates every 5 seconds on my pc), the duration (drawtime) stays similar for every frame: Could you share a recording of the Graphics process for 10 seconds? |
Most of the stutters are just caused by changes on the webpage that the YouTube design has to recalculate things for. For some you can reduce the duration. Because all 3 microstutters are partially caused by your Bitwarden extension. It is executing a Screenshots of the stutters (the red lines) |
Oh thanks i didnt know autofill would take so much. Thanks for the help! And ill be waiting for firefox v125 or 2.38.2! |
@Askejm I noticed that Firefox released version 125. Can you confirm that Mozilla has fixed the bug? |
@WesselKroos Yes works all good now |
Troubleshooter
First try all the checks and steps in the troubleshooter: https://github.com/WesselKroos/youtube-ambilight/blob/master/TROUBLESHOOT.md
No
The lower quality settings helped somewhat, although i still get around 15-20fps
Describe the performance you are currently experiencing
3. Low performance on 4K, 50% res scaling. It only happens in Firefox as of lately. No problems in Chrome.
Device information
4. Name of your CPU: 5800X3D
5. Name of your GPU: 3090
6. Name of the browser: Firefox
7. Version of the browser: 123.0
Screenshots
8. Add a screenshot of the Task manager with the CPU and GPU usage charts
10. Turn on the Ambient light setting Quality > Show framerate and make a screenshot of your framerates
Describe the performance you would be happy with
11. At least 30fps on 4k. I prefer the higher bitrate of 4k, but find the delay between scenes quite jarring. I know i can enable the fade duration, but would prefer not to
Describe any workaround you've tried so far
12. Fade duration and lower quality settings. They both mitigate it somewhat but is not something i am satisfied with.
Additional context
Add any other context or screenshots about the feature request here.
13. It only started happening a little bit ago. I am unsure if it's an automatic update, but I did recently update my driver which may have caused some issues with this plugin.
The text was updated successfully, but these errors were encountered: