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

(develop branch) Symbolic links for raw, original and compressed when using NFS share #1374

Closed
2 tasks done
anderflash opened this issue Apr 7, 2020 · 2 comments
Closed
2 tasks done
Assignees
Labels
enhancement New feature or request

Comments

@anderflash
Copy link

My actions before raising this issue

Expected Behaviour

Reduce storage by using links on raw, original, and compressed subfolders when using NFS share.

Current Behaviour

As far as I understood from engines/task.py (develop), data is copied from share to task/raw, then clustered and zipped to task/original and compressed to task/compressed, resulting in 4 times the same data.

Possible Solution

Is it possible to add an option to only use symbolic links? This is crucial when we have GBs of images and videos. Some related issues are #1210 and #204. Since folders task/original and task/compressed contain zipped chunks, it might be more difficult to mitigate, but for task/raw it's possible. There is an argument to the zip command called --symlinks, but I don't know exactly whether it's reflected onto the python ZipFile class. It's gonna be more difficult to the compressed folder, since it manipulates the images (by scaling the color range to 8 bits and converting it to 8 bits before compression).

Steps to Reproduce (for bugs)

n/a

Context

Each video has around 2 GB of size, and we have dozens of them. Each will be annotated by some students. We expect multiple users for the same dataset (in order to compare their annotations).

Your Environment

  • Git hash commit (git log -1): commit 76fc8e4 (HEAD -> develop, origin/develop)
  • Docker version docker version (e.g. Docker 17.0.05): Docker version 19.03.5, build 633a0ea838
  • Operating System and version (e.g. Linux, Windows, MacOS): Move function from closure to class method #42~16.04.1-Ubuntu SMP Tue Oct 10 16:32:20 UTC 2017
@nmanovic nmanovic added the enhancement New feature or request label Apr 7, 2020
@nmanovic
Copy link
Contributor

nmanovic commented Apr 7, 2020

@anderflash , thanks for the request. It is a critical problem. Many users complain about that and it is time to find a solution.

@nmanovic nmanovic added this to the 1.1.0 - Alpha milestone Apr 7, 2020
@nmanovic nmanovic modified the milestones: 1.1.0-alpha, 1.1.0-release Jun 22, 2020
@nmanovic
Copy link
Contributor

@anderflash , please let us know if it is an issue for us. We have optimized working with data. It should not be an issue. I will close the issue as outdated. If the issue isn't resolved, don't hesitate to reopen.

@nmanovic nmanovic removed this from the 1.1.0-release milestone Nov 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants