From 10dab8eef817d3e149ca0002ccdc32e91ab4ac82 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=B8ren=20Lind=20Kristiansen?= Date: Fri, 5 Jan 2018 09:37:05 +0100 Subject: [PATCH] Remove dummy variable from function calls --- spacy/__init__.py | 2 +- spacy/cli/download.py | 3 +-- spacy/tests/regression/test_issue1622.py | 2 +- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/spacy/__init__.py b/spacy/__init__.py index 36ff2dd1a..49701bb96 100644 --- a/spacy/__init__.py +++ b/spacy/__init__.py @@ -25,4 +25,4 @@ def blank(name, **kwargs): def info(model=None, markdown=False): - return cli_info(None, model, markdown) + return cli_info(model, markdown) diff --git a/spacy/cli/download.py b/spacy/cli/download.py index 455b839b9..36babef1c 100644 --- a/spacy/cli/download.py +++ b/spacy/cli/download.py @@ -38,8 +38,7 @@ def download(model, direct=False): # package, which fails if model was just installed via # subprocess package_path = get_package_path(model_name) - link(None, model_name, model, force=True, - model_path=package_path) + link(model_name, model, force=True, model_path=package_path) except: # Dirty, but since spacy.download and the auto-linking is # mostly a convenience wrapper, it's best to show a success diff --git a/spacy/tests/regression/test_issue1622.py b/spacy/tests/regression/test_issue1622.py index a2b9f438a..19cab3942 100644 --- a/spacy/tests/regression/test_issue1622.py +++ b/spacy/tests/regression/test_issue1622.py @@ -86,6 +86,6 @@ def test_cli_trained_model_can_be_saved(tmpdir): # spacy train -n 1 -g -1 nl output_nl training_corpus.json training \ # corpus.json - train(cmd, lang, output_dir, train_data, dev_data, n_iter=1) + train(lang, output_dir, train_data, dev_data, n_iter=1) assert True