-
Notifications
You must be signed in to change notification settings - Fork 465
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
Deleted files still appear on file list until view is refreshed #374
Comments
On feature list I see following in bugs:
Maybe it's somehow connected? But I do not see relevant bug report. |
Another update. While trying to remove a few larger files (50-200 MB) through Yazi my computer froze. I've got my nvme LED diode blinking on PC. It was reproducible - when I did it again after PC restart I had same freeze effect. It has happened to me for the first time. No other issues like that with this drive before. However, it's too little to prove it's Yazi's fault. Hence, I'm not opening new issue for now. If anyone has similar problem please report. I am on Linux Arch kernel 6.5.9-arch2-1. |
I can confirm this issue. Wayland on NixOS, kernel 6.6.x, tried different terminals (kitty, alacritty, foot), all show the same behaviour mentioned by the issue creator. I do have the issue since 1.5, 1.4 was ok for me. At the moment I use master branch and its still reproducable. After the problem is triggered, the yazi process stays above 100% in top. tasks_show inside yazi doesn't show any unfinished tasks. It never recovers from that state even if I keep running it for hours/days. Its sort of reproducable for me by moving a bunch of files to a network folder with a large number of files. It doesnt seem to make a difference if its 10 000 or 300 000 and also there is no difference in mounting the network drive via smb or nfs protocol. I still haven't figured out if its related to the folder being luks-encrypted and if it does matter if the share is local or networked. Have to do some more testing to hopefully further isolate the problem. |
@solvation thanks for confirming. In my case disk is neither encrypted nor network mounted. Last time it happened to me after I made a few delete operations, but then I had a freeze so that was a bit different. To add fs details I use btrfs. |
I've tried for a long time, but unfortunately, I couldn't reproduce it on my macOS and Linux (Kernel 6.1.12). I attempted to copy 10 video files of ~500MB each to both my internal SSD(APFS) and external USB(ExFAT). The CPU fluctuated between 0.4% to 2.2%, and there were no issues with freezing. The file list refreshed correctly. Then I tried copying a directory containing 4992 small files totaling 1.68GB. The CPU fluctuated between 3.6% to 6.4%, and it performed normally. The entire test was conducted using the latest main branch code. I'm unsure how to pinpoint this problem. Would it be possible to provide a screen recording of your steps? Also please try reducing the concurrent tasks to yazi/yazi-config/preset/yazi.toml Lines 69 to 71 in d2963b2
|
It has been addressed in the "feat: better file hover state" PR, so it's unrelated. |
Please note there may be two separate bugs (problem with listing and freezing). I've isolated one issue which I found a way to reproduce. Still, the issue with proper list refresh may be unrelated. |
I tried to debug it a bit with gdb and found out 2 things:
|
Ok, found a way to isolate the not refreshing list issue as well. This also seems to be connected with Ueberzug.
The directory won't be removed until you move out of that parent directory and in again. Then when you try to operate on other files/dirs you could notice similar problems with refreshing. |
Hi @solvation, the creator of this issue has split it into two separate problems. Let me know which problem you are currently experiencing, or them both? |
Would the issue exist without Ueberzug? i.e. using a terminal that supports displaying images by itself, and uninstalling Ueberzug. |
I just tried another attempt following the steps above, sadly I still couldn't reproduce it... CleanShot.2.mp4 |
Hi, thanks for your time! I'll try to get a working screen capture of the issue as you suggested but have to do some setup for that first. |
Example 1: deleting all files in a folder but one -> deleteHB.mp4Example 2: cutting all files in test1 and pasting them into test2 -> movingHB.mp4 |
I can't download both files, "The transfer you requested has been deleted." |
Yep, I can confirm this issue exists as well on kitty while ueberzugpp uninstalled. I've tried Wayland/X11. However, I've been playing a bit more with that issue. On the main branch from git I can also experience this bug, but it's much more difficult to reproduce it (than with 0.1.5 version) and my simple steps given above may not work so well. Right now SOMETIMES when I'm moving through the list of photos and then I try to delete a photo the view is not refreshed. Although, it's kind of race condition, sometimes it happens, but not always. Probably I can't give very clear steps for reproducing. I just give a video so you will have an idea of what I'm talking about. output1.mp4 |
Funny enough. I've discovered this bug appears even without ueberzug working on alacritty. Hence, maybe it's even not connected to preview. And actually this pattern I've shown on the movie is quite reproducible.
|
Hi I made a PR to fix it, #381, let me know if it works for you! pr-381.mp4 |
Not the OP but I just checked your PR with
Sorry to say but I can still reproduce the problem on my machine though. I managed to debug a little bit further and it seems that, once yazi gets in that "bad state" where it doesnt properly refresh, it keeps creating [file] processes and immedeatly kills them. Even if I dont do anything in yazi and I'm p.e. in an empty dir where absolutly nothing should be "to do" for it. |
Thanks for your feedback @solvation. It's possible that your issue is different from the OP, or it might be that this PR isn't effective at all. I'll wait a bit for the OP. |
Yeah, plenty of changes in this PR. Great job! I've been trying this PR for a while and I don't experience issues mentioned in this ticket. If anything changes I'll get back to it. PS. There are some second order issues. As this is development branch I am not creating new tickets.
|
Thank you for testing; glad to see the issue has been addressed. I'll merge this PR now!
This is intentional. Evaluating directory size is a time-consuming operation involving a large amount of I/O operations. Ideally, watching all file changes within a directory recursively (not just deletions, but also creations and modifications that affect directory size) would be preferred. However, this isn't an efficient approach. For performance and stability reasons, Yazi limits watching to a depth of 2 levels (having too many levels would exceed the file descriptor limit). Another method is to allow users to manually refresh the size by pressing a key, but this would be an enhancement, not a bug.
I need more reproducible details. Would you like to file an issue for it?
This is a known bug and should be fixed soon.
I haven't encountered this issue yet, I'll pay attention and observe. If you find any clues to reproduce it, please create an issue so I can further investigate! Hey @solvation, it seems like you're experiencing a different issue from the OP. Would you like to file a separate issue for it? |
Done! Fixed in #382 |
I'm going to lock this issue because it has been closed for 30 days. ⏳ This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
What system are you running Yazi on?
Linux Wayland
What terminal are you running Yazi in?
Alacritty
Yazi version
0.1.5
Did you try the latest main branch to see if the problem has already been fixed?
Not tried, and I'll explain why below
Describe the bug
Sometimes while doing
yank --cut
/yank
,paste
,remove
the file list does not seem to be properly updated. For instance I have executed remove command, but the file still appears on the list. Only after refreshing directory list (moving out and entering directory again) the list is updated.Expected Behavior
Removed files should not appear any longer on file list. Pasted files should appear on file list.
To Reproduce
I don't have clear steps to reproduce this behavior, but I will try to be attentive and keep you updated. My guess is, this behavior may be connected to some unfinished task hanging. While this happened I: (1) opened multiple times helix from yazi or some files with xdg, (2) yanked files, cut, pasted or deleted, (3) created new directories.
Configuration
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: