swww icon indicating copy to clipboard operation
swww copied to clipboard

fix vertical monitor transitions & 'failed to load cache' errors

Open Nynxz opened this issue 11 months ago • 4 comments

Fixed Issues:

  • Transitions on vertical monitor's were having issues
  • [WARN] failed to load cache: there is already another swww process running errors on wallpaper changes

Discussion: Wallpaper changing animations do not work on vertical monitor

have not written rust, I am not sure if this causes issues with https://github.com/LGFae/swww/commit/56928f7c0e55a321ad5a83a039e0e0dc3eeac35b

Nynxz avatar Jan 17 '25 15:01 Nynxz

I believe the problem with this is that it wouldn't reload the image if the output's configuration changed. That was the original reason for that commit.

LGFae avatar Jan 17 '25 16:01 LGFae

I also do not understand how this could possibly be the solution to #386. There seems to be a more fundamental issue we have to address.

LGFae avatar Jan 17 '25 16:01 LGFae

https://github.com/user-attachments/assets/da9f0e5f-f948-469e-9e09-abf96dae35ad

Not sure, the issue did not exist before https://github.com/LGFae/swww/commit/56928f7c0e55a321ad5a83a039e0e0dc3eeac35b I feel it would be better to remove the problem code and try fix what was intended by https://github.com/LGFae/swww/commit/56928f7c0e55a321ad5a83a039e0e0dc3eeac35b (refreshing when monitor input changes) a different way over trying to fix the issue https://github.com/LGFae/swww/commit/56928f7c0e55a321ad5a83a039e0e0dc3eeac35b caused. I.e listening to actual wayland events, rather than constantly polling for whether the 'surface has changed', which is what it seems like its doing currently, and causing a memory leak.

there is also a memory leak if transitions get overwritten midway through (changing the wallpaper too quickly) but that is a separate issue to these ones.

Nynxz avatar Jan 26 '25 16:01 Nynxz

It seems to be working now (after upgrading to Hyprland 47), so I closed the original. Unfortunately so many things broke in hyprland 47 for me that I will be switching back to 46 for now, meaning I have to live with the animations.

GeodeArc avatar Jan 29 '25 06:01 GeodeArc

Alright, we may have the proper fix for this in PR #439, actually.

LGFae avatar Jun 06 '25 14:06 LGFae