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

Poor performance with tramp #8

Open
mavit opened this issue Jun 1, 2017 · 4 comments
Open

Poor performance with tramp #8

mavit opened this issue Jun 1, 2017 · 4 comments

Comments

@mavit
Copy link

mavit commented Jun 1, 2017

I found all-the-icons-dired to be unusably slow on a remote tramp directory containing a few hundred files. Commenting out the following line improved performance a great deal:

	(if (file-directory-p filename)
@domtronn
Copy link

domtronn commented Jun 2, 2017

I faced a similar issue as part of spaceline-all-the-icons here

You can use the following to decide whether or not a file is tramped and optimise around that maybe?

(file-remote-p filename)

Unfortunately, that doesn't really help solve the problem as I guess you'll still want to know whether or not the file is a directory and it's still costly to run any of the file commands on tramp files! 😅

I'm not sure whether f.el performs any better... But ideally, you'd want to interface with tramp

seagle0128 added a commit to seagle0128/.emacs.d that referenced this issue May 19, 2019
@seagle0128
Copy link

I think file-remote-p might be helpful, but if not, try file-local-name. It works well for me.

@mtekman
Copy link

mtekman commented Jul 17, 2019

I am seeing this slowdown also.

Is there a annotate-folder-names-with-icons hook where we can count the number of items in the dired buffer? We could then make a decision on whether to render based on that.

@dhyansraj
Copy link

I was getting slow performance and fixed by running loop unless it is a remote directory. I faced the same issue with Centaur Emacs too. There the suggestion was to advise projectile, but that didnt help much. But this fix makes it much faster.

(defun all-the-icons-dired--refresh ()
  "Display the icons of files in a dired buffer."
  (all-the-icons-dired--remove-all-overlays)
  (unless (file-remote-p default-directory)
...
...

seagle0128 added a commit to seagle0128/.emacs.d that referenced this issue May 27, 2020
seagle0128 added a commit to seagle0128/.emacs.d that referenced this issue May 27, 2020
arouene pushed a commit to arouene/all-the-icons-dired that referenced this issue Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants