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

Allow spacy.en.download to take a file path #637

Closed
ExplodingCabbage opened this issue Nov 18, 2016 · 4 comments · Fixed by #642
Closed

Allow spacy.en.download to take a file path #637

ExplodingCabbage opened this issue Nov 18, 2016 · 4 comments · Fixed by #642

Comments

@ExplodingCabbage
Copy link
Contributor

As noted at #208 (comment), it's possible to install spaCy models to a custom path via sputnik.install, but looking at the code for en/download.py it seems that there's no way to specify a custom path when installing via python -m spacy.en.download.

It would be nice to have this feature, especially since download.py offers niceties like checking whether the model is already installed that sputnik.install doesn't do.

A syntax like python -m spacy.en.download --download-path=/path/to/download/model/to seems like it'd be ideal.

@honnibal
Copy link
Member

Sounds great. Do you want to make the pull request?

@ExplodingCabbage
Copy link
Contributor Author

Sure - I'll try to get round to it today, otherwise I'll put something together over the weekend.

@honnibal
Copy link
Member

Btw, I've been trying to improve options for file loading more generally. There's some stuff in spacy.utils that isn't in the docs yet but might be relevant. Specifically, you should be able to set the default data path.

@lock
Copy link

lock bot commented May 9, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants