-
-
Notifications
You must be signed in to change notification settings - Fork 585
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
[Feature Request] Need help testing animations #1052
Comments
Hi there, As far as i can tell, The quality of the animations are pretty neat, great job on that. Overall this is probably the best looking animation fork of picom i ever tried. Thank you for your work. I will spin a few more wm and report back if i ever find something off. Edit : resizing windows seems to massively wobble the inner content and create lots of visual artifacts. Here's the config used :
|
Hello. I tried it on The animation quality is fairly good. But it can give you a headache for some reason. I think it would be right to exclude popups from animation by default, as @Amoebawp said. Even the right-click popups on browsers fly across the screen for me. Also, thanks to @Amoebawp for the config. I would've struggled a lot searching for it otherwise. |
I tried with Xfce, it worked. I used the configs for animations by @Amoebawp. Tomorrow I will give another look at options. |
I tested it, so far I can't use it because too many crashes.
or
Here's a testing video (sorry for the quality, I was on my shitty laptop) P.S. I can also test on my NVidia GPU. P.P.S. I used configuration provided by @Amoebawp |
I didn't see any crash. I use the backend "egl" and start picom with -b (daemonize). |
Hey everyone, Appreciate your time for trying to test it. I've made a new commit, could you please try the latest one? I hope this will work now. @Monsterovich that is another issue I think, for fading to work on animations I needed to add unmapping statement there. |
I hope im not thinking correctly because this will work with window managers or DE but not both |
Hi, tried with commit
|
Tried the latest commit on dwm, i3, bspwm, leftwm and awesome. All were working as expected really, some minor visual bugs - when opening several windows, some may glitch a bit and fail to render any animation. It's not particulary annoying. Here's a quick capture on DWM, but all where behaving like this on my end If i had to mention two annoying things : visual glitch when resizing window + animations on right click/context menu (Xmenu) that would need to be excluded. @FT-Labs do you need us to test something a bit more specific ? Here's an updated config after some adjustments to my liking ( with Fading )
|
bspwm user here searching for fork with animations, rounded corners and blur, haha. for me, all works cool except animation-for-prev-tag, it only executes once. Thats it, everything else is flawless. edit: actually, https://github.com/fdev31/picom works flawlessly. Only thing is i'd like workspace switch animations. |
Tag change animations don't work in awesome, it looks like it doesn't recognize it as a tag change? Also when minimizing a client the animation for it doesn't play at all. |
actually, it doesnt now. strange one, prev-tag executes only once as in ft-labs. needs testing |
@BANanaD3V It's not next-tag or prev-tag animation. It's window creation animation. It execute once because picom think that it's newly created window. |
Tag switch will be fixed soon. The other issue that is scratching my mind is properly handling xunmapwindow and xdestroywindow events. Some simpler window managers like dwm do not use minimize or destroy windows. It only unmaps the window and free the window because there is no iconization or minimization in dwm. After I got the fix for this issue, hopefully the last issue is going to be fixing glitches while shadows are enabled on low performance computers. |
It crashes when I close a window on qtile with the following error : |
My 2 cents on i3 : It works, it's usable but not optimal and still sometimes feels glitchy, it's an overall feeling, I don't know how to express it clearly but for example when refreshing my config polybar may not appear and I would need to refresh again, not a real world case but just for reference if I spam super + enter and open bunch of terminals the desktop will quickly become ultra zoomed without any possibility to take control other than going to a tty and kill picom, with main picom I feel like I could open the double amount of terminals easily without much problems. |
Every animation is extremely buggy (not usable) with the following error : |
Tested on Openbox. Visual glitches with the title bar when minimizing regardless of effect (but like 10% of the time it works fine?) and some windows like Discord and Firefox will animate the first time they're minimized and opened but not again. Other than that, it works great. My config:
|
Working amazingly on BerryWM. Only problem is the occasional lag. Here's a video: animations_1.mp4 |
Can we have an update with latest patches on this fork of picom, I am using it and it has a lot since I installed it, related to the yshui/picom. |
Qtile user here! I tried your fork and it works really well EXCEPT the tag animation 2023-08-08.16-12-45.mp4Thats my only problem. I also saw other comments that says basically the same issue so yeah |
Will update this soon! Thanks for the response guys |
Also using Qtile but my issue might not be related to the desktop used. It doesn't seem like animation set for transient windows (Dunst notifications in my case) is respected Is it missing something on my end? $ notify-send "Test"
$ xprop
The notification is using the animation from |
@angrybacon in mean time try
|
Thanks for the workaround. I notice something else just now, the Is it expected? If so, are there plans for |
Animations for prev tag and next tag still don't work on awesomewm right? |
Nope |
Working on it rn |
Configure Animations Specifically for Window MaximizationI've noticed that the current animation settings are not restricted solely to window maximization; they're also being applied to standard window operations. This isn't the desired behavior. When performing routine tasks such as moving, resizing, or simply focusing on a window, these animations are triggered. This not only results in a visual distraction but also slows down the overall user experience, especially when working with multiple windows simultaneously. Ideally, I'd like the option to configure animations to be exclusive to certain actions, like window maximization. This would help users like myself who appreciate some animations (like maximizing) but find them cumbersome when applied universally across all window actions. |
I'm inclined to agree with @raven2cz, some things being animated doesn't make much sense. Resizing and moving windows (manually, at least) is made somewhat sluggish by animations being applied. Additionally, one weird thing I encountered using your fork is this: 2023-09-30.11-31-42.mp4Context menus seem to be the "same" window when closing and re-opening them, resulting in an animation of their position when re-opening them. The same applies to tooltips. |
Does this happen on firefox only, or with all context menus? It could be that firefox (or other apps too) simply hide the context menu and show it when you right click again, and only then they update the position, which would explain this behavior. I hope its possible to not animate the context menu specifically. |
Indeed it doesn't happen in every app. Happens in Nemo, but not in Nautilus, not in Dolphin, not in Lutris, not in Chromium...
That makes total sense. I'm not sure it's the case in this fork but in some other picom forks there was a way to disable animations for certain window types. |
Hello, Guys just as a sidenote, some of the animations can be closed. There is an animations_exclude property on this fork. wintypes: Just set animations to false on the window type you want. It would disable the animations. |
It's not really about the types of windows (clients) here. If we think about it, it's purely about disabling animations primarily for manual window control via mouse only. In something like Awesome WM, it might be possible to assign a status or environment variable that the environment can read. Awesome WM is highly customizable since it's a framework. Just let us know what's needed and we can supplement. Besides, I still don't understand, if I wanted animation for a specific task like maximizing, why are there active animations for window movement? Isn't Picom then poorly designed? After all, it's not a problem to provide additional information for Xorg, so why not for Picom too? I can help with awesome. We've been waiting for years for Picom's behavior to be corrected. |
@FT-Labs I've been testing the fork after it working perfectly in BerryWM, but after switching to AwesomeWM, all animations, especially tag switching animations, are completely broken. Instead of sliding from one side to the other, they just either fade or don't work at all: 2024-01-24.09-30-54.mp4Also, just like @Vermoot, firefox popup menu animation is completely broken. Using latest git from your fork and this config |
Some feedback after using the fork for a while: I was using FT-Labs@b73e6a0 for a few months and it's been working great. The other day I updated to FT-Labs@fe5b416. Immediately, I could tell the animations looked better. I've not closely inspected what exactly changed, but it just "felt better" in general. UPDATE: I can reproduce the performance issue mentioned below on upstream picom, and so it should not be attributed to @FT-Labs animation fork.Performance issue
At the same time however, I did notice a performance regression with unexpected high CPU usage. It is triggered by "animations" in the browser such as:
The high CPU usage is not one core being maxed out forever. Instead, two cores take turns getting maxed out one after the other forever (or until the animation stops).
|
Did the workspace sliding animation ever get resolved for Awesome? |
@Tensaiz (not really an answer to your question, but generally) unfortunately there will never be a way to do workspace animation reliably on picom's side. So I don't plan to implement that in picom (#1253). OTOH, I might add a way to trigger an animation externally (e.g. |
@yshui this is one of the most important animations IMO :/ Maybe make Picom a GObject Introspection to add a built-in compositor to the window manager? I use Awesome btw |
With awesome you should be able to make a dbus request to picom before workspace transition, so you should be able to start animation this way. |
hey, #1253 adds some basic animations to picom. can you give that a try and give me some feedback? |
Note Posting here since my issue only happens with the fork. Since a couple months or so (sorry I can't recall a more precise time period) it seems Picom is ramping up memory and CPU usage. At the time of writing this comment, $ fastfetch
OS: Arch Linux x86_64
Kernel: Linux 6.9.5-arch1-1
Uptime: 2 days, 4 hours, 5 mins
Packages: 917 (pacman)
Shell: zsh 5.9
Display (MD34PRO): 3440x1440 @ 60Hz
DE: qtile
WM: LG3D (X11)
Terminal: kitty 0.35.1
CPU: Intel(R) Core(TM) i5-6500 (4) @ 3.60 GHz
GPU: AMD Radeon RX 6600 XT @ 1.02 GHz [Discrete]
Memory: 9.12 GiB / 15.54 GiB (59%)
Swap: 1.22 GiB / 4.00 GiB (30%)
Locale: en_US.UTF-8 When that happens, I usually kill For reference, here is the configuration I'm using it with https://github.com/angrybacon/dot/blob/9eed042f28e46615dcc2c8e7d484da074a6dd019/config/picom.org |
@angrybacon if you know how to use perf, it'd helpful to know where CPU time is spent. For memory usage I recommend building picom with sanitizers to detect leaks. Otherwise try to figure out how to reliably reproduce this problem, so I can investigate. |
@angrybacon hmm, this is interesting but doesn't pinpoint the problem. unfortunately your mesa libraries are stripped so I don't know what function is using the CPU. you need to recompile your mesa without stripping, and ideally also enable debug info and frame pointers. |
nice animations, though very laggy. the lag is very random. sometimes(compiling stuff) at 99% can still run smoothly. but sometimes(under 1-5% load) can be laggy. I am not sure how the animations work. But it's definatly not using gpu. Since my cpu usage went up during the animation, and not gpu usage(im using nvidia) |
Hello guys,
I did not want to open up a pull request since it might have some way to go for it.
I've tried to implement a general animation fork, with minor code added. It is on a different branch, I will post the link below.
I would appreciate it if someone can try with different window managers, I've only tested this on pdwm only.
Thanks for everyone who wants to help.
https://github.com/FT-Labs/picom/tree/generalanimation
The text was updated successfully, but these errors were encountered: