Skip to content
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

[4.3.beta1] Editor becomes slow to unusable when running a separate plugin-based window next to it #93169

Closed
SnekOfSpice opened this issue Jun 14, 2024 · 5 comments

Comments

@SnekOfSpice
Copy link

SnekOfSpice commented Jun 14, 2024

Tested versions

  • Reproducable in v4.3.beta1.official [a4f2ea9]
  • Does not occur in v4.3.dev5.official [89f70e9] or v4.3.dev6.official [89850d5]

System information

Godot v4.3.beta1 - Windows 10.0.19045 - GLES3 (Compatibility) - AMD Radeon RX 5700 XT (Advanced Micro Devices, Inc.; 31.0.12027.9001) - AMD Ryzen 7 3700X 8-Core Processor (16 Threads)

Issue description

Starting with 4.3.beta1, a plugin I've been working on became unusable. It creates a separate window that the user will use like its own application (see mrp equivalent image below). Something in beta1 makes this setup perform really really badly.

grafik

Here I only added a small slider and some code from my own codebase to give the editor something to struggle with, as this does seem to scale with scene complexity. (I couldn't reproduce this with an empty window, with the setup above, it took about 2 minutes until this drop in performance was reached, and with my full plugin the editor can barely switch script tabs in under 30 seconds)

  • 0:03 - the hover effect of the different menu items breaks
  • 0:14 - the different tab buttons all have a delayed hover effect
  • 0:20 - drag-clicking to highlight console text breaks
  • 0:52 - double clicking on console text to highlight it works unreliably
  • 1:00 - windows open and close with heavy delay
2024-06-14_17-33-05.mp4

While looking for related issues I came across #93155, which seems to be the only other editor performance-related issue for 4.3.beta1 so far?

Steps to reproduce

  • Import the project and ensure the plugin is enabled (reload the editor if needed)
  • Open the window with the "Test" Button in the top right, kill some time by playing with the slider or just wait around
  • After some time, the main Godot editor should become laggy like in the video
  • (This only happens when the window is open, if you never click the button, the editor is fine)

Minimal reproduction project (MRP)

beta-1-bad-performance.zip

@JekSun97
Copy link

JekSun97 commented Jun 14, 2024

I confirm that productivity drops greatly.
I noticed that when we comment out window.popup() -> #window.popup(), the editor stops freezing, something with laggy_window.

Also, after clicking the "Test" button, the following errors are displayed:
err

@matheusmdx
Copy link
Contributor

matheusmdx commented Jun 14, 2024

This seems to be another victim of #91361, you don't need to wait for reproduce the issue, just move the opened window a bit and will be enough to trigger the lag

unknown_2024.06.14-18.47_1.mp4

Also as the other similar issues, pr #92742 seems to solve this issue too, but just to make sure, @SnekOfSpice can you check this comentary (#92954 (comment)) and do the steps to confirm?

unknown_2024.06.14-18.48.mp4

@SnekOfSpice
Copy link
Author

Thanks for the pointer @matheusmdx! This does indeed seem to fix the issue. Closing this issue then :)

@matheusmdx
Copy link
Contributor

@AThousandShips This is the third issue opened after the fix, at this point shouldn't be a good idea let one of these issues open and pin them?

@AThousandShips
Copy link
Member

All of the issues linked in the PR have very very different descriptions so I don't think people will identify based on the issue if we pin it (and pinned issues aren't very effective either, for example the C# export issue has two duplicates opened after it was pinned)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants