You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Because LocalFileSystem.info returns t: 'link' for sym linked files / directories, isdir will always be false. This is inconsistent with the behaviour of os.path.isdir which follows sym links. We've been able to fix this by overriding isdir to just return os.path.isdir. I guess that this behaviour isn't intended? Not sure what the best fix is, overriding isdir works but is not the prettiest solution 😃
The text was updated successfully, but these errors were encountered:
I think it would be fair to follow whatever os does. It will mean breaking the weak requirement that info["type"] be "directory", but I'm not too worried about that.
We had a bit of a subtle bug here: Lightning-AI/pytorch-lightning#6730
Because
LocalFileSystem.info
returnst: 'link'
for sym linked files / directories,isdir
will always be false. This is inconsistent with the behaviour ofos.path.isdir
which follows sym links. We've been able to fix this by overridingisdir
to just returnos.path.isdir
. I guess that this behaviour isn't intended? Not sure what the best fix is, overridingisdir
works but is not the prettiest solution 😃The text was updated successfully, but these errors were encountered: