| Summary: | WindowHeap-based effects are slightly slower to activate than previous versions | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Phos <phyllon> |
| Component: | effects-window-management | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | CLOSED FIXED | ||
| Severity: | minor | CC: | al.neodim, bugseforuns, faf04348, fanzhuyifan, linx.system.adm, liubomirwm, makosol, nate, notmart, polardelphine, postix, tjkrobertson, unblended_icing552, vlad.zahorodnii, wbqq |
| Priority: | HI | Keywords: | regression |
| Version: | 6.1.2 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| See Also: |
https://bugs.kde.org/show_bug.cgi?id=459467 https://bugs.kde.org/show_bug.cgi?id=476002 |
||
| Latest Commit: | Version Fixed In: | 6.0 | |
| Sentry Crash Report: | |||
| Bug Depends on: | |||
| Bug Blocks: | 490178 | ||
|
Description
Phos
2022-06-22 10:21:34 UTC
Can confirm. It's become faster recently, but the small delay is still there. *** Bug 459281 has been marked as a duplicate of this bug. *** *** Bug 453221 has been marked as a duplicate of this bug. *** It seems to have gotten worse again in 5.26. The new animation curves are nice though. Still a very noticeable lag between pressing the button and the animation. Another issue with 5.26 is that the animation never plays again when you activate the effects multiple times. For example, the Present Windows effect just doesn't play when you press the activation button multiple times. It takes the animation so long to activate that I have already released the key. Its pretty jarring how slow it is to activate. *** Bug 462608 has been marked as a duplicate of this bug. *** On a 6th gen i5 it takes around 4 seconds to activate the overview effect the first time, however, that only happens the first time. All subsequent activations work fine. I wonder if it would be possible to preload the active effects when the session starts. *** Bug 465419 has been marked as a duplicate of this bug. *** *** Bug 469446 has been marked as a duplicate of this bug. *** I wish devs deliver previous version of the present-windows but they responded on reddit that they won't. New present-effects version is a complete regression in terms of perfromance, usability and configurability. If anyone is interested in compiling old version please let me know. I think this also affects the Application Launcher. This is a very big issue, waiting for 1-3 seconds before it appearing... ... And tooltips, it basically makes the taskbar freeze until the effect starts functioning. Lyubomir, none of those issues are related; this bug report is only about WindowHeap-based KWin effects, which do not include Application Launcher or Plasma panels. Please submit new bug reports for new issues. (In reply to Lyubomir from comment #12) > I think this also affects the Application Launcher. This is a very big issue, waiting for 1-3 seconds before it appearing... I don't use Application Launcher so I can't confirm or disprove this. I use desktop grid and it works perfectly. Quite fast. *** Bug 455585 has been marked as a duplicate of this bug. *** A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/4768 Git commit f1ff706b3b1e769eaf67c4a5042e522edc286ab3 by Vlad Zahorodnii. Committed on 08/12/2023 at 11:21. Pushed by vladz into branch 'master'. scripting: Cache WindowThumbnail textures Effects like overview can create more than one thumbnail for the same window. For example, if you have 4 virtual desktops, the overview is going to create 9 window thumbnails for the desktop background, which in its turn means that each thumbnail is going to have its own texture. That's not great. WindowThumbnailItem.sourceSize has been dropped because it's unused and will complicate texture caching. M +156 -147 src/scripting/windowthumbnailitem.cpp M +37 -18 src/scripting/windowthumbnailitem.h https://invent.kde.org/plasma/kwin/-/commit/f1ff706b3b1e769eaf67c4a5042e522edc286ab3 Git commit eb13085de2173cb31dbb645725462d5cefadb213 by Vlad Zahorodnii. Committed on 08/12/2023 at 14:40. Pushed by vladz into branch 'master'. plugins/overview: Load screen delegate asynchronously The compilation step still takes a significant amount of time on the first launch. On my machine, it's around 1s. Even if it happens just once, freezing the session for 1 second is not great. This change makes the overview effect load main.qml asynchronously when plasma session starts. By the time the session is loaded, it should be ready. M +4 -0 src/effect/quickeffect.cpp M +8 -1 src/plugins/overview/overvieweffect.cpp https://invent.kde.org/plasma/kwin/-/commit/eb13085de2173cb31dbb645725462d5cefadb213 Git commit ef2c4e201e2f062798068b728066630bf5ff85de by Vlad Zahorodnii. Committed on 12/12/2023 at 16:21. Pushed by vladz into branch 'master'. plugins/overview: Load milou on demand Milou.ResultsView loads runners regardless whether the query string is empty. It's not clear what milou should do. There are valid arguments both in favor and against preloading runners. This change puts Milou.ResultsView behind a Loader so milou is loaded when it's actually needed and not when starting overview. M +10 -5 src/plugins/overview/qml/main.qml https://invent.kde.org/plasma/kwin/-/commit/ef2c4e201e2f062798068b728066630bf5ff85de https://invent.kde.org/plasma/kwin/-/commit/0ff4f84aceef4262d4da3602d3cbe34c630ea2de should help too. This is massively improved for me now; well within the range of the snappiness of the old C++ based effects for me, even when the system is under load. Let's call it fixed now! After a recent update, I can experience the same symptoms. Present windows is only activated with a very noticable delay. When you try to spam "meta+w" it is super laggy and easyly reproducable through that way. I have kwin version 6.1.2-1, and I can report the same Issue again. Task switcher and present windows are having some noticable amount of delay before they appear/function. > After a recent update This would suggest there was a regression unrelated to the original issue. Please open a new bug report for it. > Task switcher This is a different issue, likely Bug 479250. |