Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Commit

Permalink
Keep NLTK down. They broke the download of omw. (#5540)
Browse files Browse the repository at this point in the history
  • Loading branch information
dirkgr authored Jan 13, 2022
1 parent 73a5cfc commit a91946a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ DOCKER_RUN_CMD = docker run --rm \
# These nltk packages are used by the 'checklist' module. They are downloaded automatically
# if not found when `checklist` is imported, but it's good to download the ahead of time
# to avoid potential race conditions.
NLTK_DOWNLOAD_CMD = python -c 'import nltk; [nltk.download(p) for p in ("wordnet", "wordnet_ic", "sentiwordnet", "omw-1.4")]'
NLTK_DOWNLOAD_CMD = python -c 'import nltk; [nltk.download(p) for p in ("wordnet", "wordnet_ic", "sentiwordnet", "omw")]'

ifeq ($(shell uname),Darwin)
ifeq ($(shell which gsed),)
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
"cached-path>=1.0.2,<2.0.0",
"fairscale==0.4.4",
"jsonnet>=0.10.0 ; sys.platform != 'win32'",
"nltk",
"nltk<3.6.6",
"spacy>=2.1.0,<3.3",
"numpy",
"tensorboardX>=1.2",
Expand Down

0 comments on commit a91946a

Please sign in to comment.