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

Added Directory for Fast Bi-layer Neural Synthesis of One-Shot Realistic Head Avatars #43

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Bharat-mtr
Copy link
Contributor

This is regarding Issue #41
Added Directory for Fast Bi-layer Neural Synthesis of One-Shot Realistic Head Avatars

Paper :- https://arxiv.org/pdf/2008.10174
Repository:- https://dagshub.com/Bharat-mtr/bilayer-model

Fixes #41

@Bharat-mtr
Copy link
Contributor Author

@nirbarazida Please review and let me know if I need to add/edit anything

@nirbarazida
Copy link
Collaborator

Hey @Bharat-mtr,

Thanks for taking the time and contributing to our challenge!
We have two comments before we can merge the PR:

  1. The data folder is missing the files. It might be the case where DVC didn't push all the files under that directory - please run the dvc push command again and make sure all files were pushed.
  2. Enhancement [not a must]: If you can add instructions on how to load the model and test it, it would be very valuable for the community

@Bharat-mtr
Copy link
Contributor Author

Hey @Bharat-mtr,

Thanks for taking the time and contributing to our challenge! We have two comments before we can merge the PR:

  1. The data folder is missing the files. It might be the case where DVC didn't push all the files under that directory - please run the dvc push command again and make sure all files were pushed.
  2. Enhancement [not a must]: If you can add instructions on how to load the model and test it, it would be very valuable for the community

Actually the data file was having around 1.13 Million files , and that's because i was unable to upload it. The best i can do here is to provide the link of data uploaded on 3rd party website and remove the data folder from this repo.

Please let me know if you have any other suggestion or should i go with the above?

Thanks.

@nirbarazida
Copy link
Collaborator

Hey,

What is the size of the data folder?
Also, why were you unable to upload it to DagsHub? If you hit an error, can you please share it here?

@Bharat-mtr
Copy link
Contributor Author

Hey,

What is the size of the data folder? Also, why were you unable to upload it to DagsHub? If you hit an error, can you please share it here?

Actually the size of data was around 8GB , i was not getting any error just it will push 1.13M files of total 8GB then it probably would take me around 1.5 - 2 days as it was showing.

it takes time around 6 hours to run command dvc add data
then it will take around 2 days to get the dvc push -r origin command to run.

@nirbarazida
Copy link
Collaborator

Hey,

I'm not sure what is your upload speed, but with 7 MB/sec it should take you ~3 hours to upload the entire dataset.
Regarding adding them to DVC, it is hard for me to calculate the time it takes to add all files, but extrapolation from previous experience, 6 hours sounds too long.

Please contact our team on the Hacktoberfest channel on Discord and describe the problem you're facing.

If we will not manage to overcome this challenge, you can upload a subset of the dataset to DagsHub, which holds examples of all types of data, and on the readme file link to external storage where the entire dataset is stored.

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.

Claim :- Fast Bi-layer Neural Synthesis of One-Shot Realistic Head Avatars
3 participants