Skip to content

Conversation

@cs-mshah
Copy link
Contributor

@cs-mshah cs-mshah commented Mar 8, 2023

This solves #31. The advantage of having symlinks (ln -s) is that we can just reference the dataset stored at some other place and create a folder structure at any/multiple desired locations as required by some codebases. Deleting symlinks doesn't delete the original dataset.

The changes have been made to support backward compatibility of move=True and move=False along with the addition of move='symlink'. All tests have successfully passed.

@a1302z
Copy link

a1302z commented Apr 5, 2023

This works wonderfully. Instead of cloning the entire dataset now, I just have links to the original image, which saves so much time and memory. I am very much in favor of merging this into the package.

@catevita
Copy link

catevita commented Dec 7, 2023

hi, this option would be amazing, will it be integrated?

@jfilter jfilter merged commit 28efbaf into jfilter:main Jan 27, 2026
@jfilter
Copy link
Owner

jfilter commented Jan 27, 2026

Merged into main — thanks @cs-mshah! The symlink feature is now available via move='symlink'. I also refactored the duplicated copy/move/symlink logic into a shared helper and cleaned up the code with ruff.

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

Successfully merging this pull request may close these issues.

4 participants