explosion/spaCy
1
1
Fork 0
mirror of https://github.com/explosion/spaCy.git synced 2024-12-25 17:36:30 +03:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Add --always-link flag to cli.download (see #3435)"

Browse Source 
This reverts commit 583a566843.
This commit is contained in:
Ines Montani 2019-03-20 01:03:40 +01:00
parent 6cfbb2d34e
commit 685fff40cf
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
spacy/cli/download.py
View File

@ -19,10 +19,9 @@ msg = Printer()
@plac.annotations(
model=("Model to download (shortcut or name)", "positional", None, str),
direct=("Force direct download of name + version", "flag", "d", bool),
always_link=("Always create symlinks, even if not shortcut", "flag", "l", bool),
pip_args=("Additional arguments to be passed to `pip install` on model install"),
pip_args=("additional arguments to be passed to `pip install` on model install"),
)
def download(model, direct=False, always_link=False, *pip_args):
def download(model, direct=False, *pip_args):
"""
Download compatible model from default download path using pip. Model
can be shortcut, model name or, if --direct flag is set, full model name
@ -49,7 +48,7 @@ def download(model, direct=False, always_link=False, *pip_args):
# Only create symlink if the model is installed via a shortcut like 'en'.
# There's no real advantage over an additional symlink for en_core_web_sm
# and if anything, it's more error prone and causes more confusion.
if model in shortcuts or always_link:
if model in shortcuts:
try:
# Get package path here because link uses
# pip.get_installed_distributions() to check if model is a