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

Keep NLTK down. They broke the download of omw. #5540

Merged
merged 2 commits into from
Jan 13, 2022
Merged

Keep NLTK down. They broke the download of omw. #5540

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
001ebaf
Keep NLTK down. They broke the download of omw.
dirkgr Jan 12, 2022
037e775
Merge branch 'main' into nltkomw
dirkgr Jan 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Makefile
View file
Open in desktop
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
View file
Open in desktop
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