Skip to content

Follow 301 redirects in the bash script that downloads the models#144

Open
jkremser wants to merge 1 commit intojcjohnson:masterfrom
jkremser:redirect-dl
Open

Follow 301 redirects in the bash script that downloads the models#144
jkremser wants to merge 1 commit intojcjohnson:masterfrom
jkremser:redirect-dl

Conversation

@jkremser
Copy link
Copy Markdown

Fixes #143

@jspitzen
Copy link
Copy Markdown

Isn't the underlying issue here that the files are no longer accessible through http, but only https? imo, altering the BASE_URL to BASE_URL="https://cs.stanford.edu/people/jcjohns/fast-neural-style/models/" is a better solution than following the 301

@jwittenbach
Copy link
Copy Markdown

I can confirm that simply switching the base URL to use HTTPS instead of HTTP does work. However, updating the curl commands to follow redirects is probably also good in case the files move again.

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