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

Directly download url #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

FantasyJXF
Copy link

Directly download url

Directly download url
@Tvde1
Copy link

Tvde1 commented Jun 18, 2019

Why?

@FantasyJXF
Copy link
Author

At first, I couldn't download the model with the script, it took me some time to get the right url form the code.

This PR just want to help those who might have the same problem, and He or She can directly download the model with the url and wget or sth.

@Tvde1
Copy link

Tvde1 commented Jun 18, 2019

This is the XY problem. Instead of a dirty fix, try figuring out why the download didn't work.

@FantasyJXF
Copy link
Author

Not sure the reason, may be the network delay. I think it could be a available option.

@arijoon
Copy link

arijoon commented Jul 1, 2019

Adding a comment with the download link in code won't really help, anyone can just put a print statement and get it, maybe add it to README instead under FAQ or something, e.g. if model download failed, manually download from xyz

@FantasyJXF
Copy link
Author

FantasyJXF commented Jul 1, 2019 via email

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.

3 participants