Skip to content

Commit

Permalink
🔥 (Spider): remove unused nltk downloading
Browse files Browse the repository at this point in the history
  • Loading branch information
clementjumel committed Dec 20, 2023
1 parent bc32d19 commit a0c6087
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
4 changes: 0 additions & 4 deletions lib/spider/spider/in_context_learning.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@

import clearml
import hydra
import nltk
import torch
import transformers
import wandb
Expand All @@ -25,9 +24,6 @@
from spider.evaluation import build_foreign_key_map_from_json, evaluate
from spider.utils import extract_sql, is_valid_sql

nltk.download("punkt")


OUTPUT_FILE = DATASET_PATH / "output.txt"
gold = DATASET_PATH / "dev_gold.sql"
db_dir = DATASET_PATH / "database"
Expand Down
3 changes: 0 additions & 3 deletions lib/spider/spider/vanilla_llm.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
import os

import clearml
import nltk
import torch
import transformers
from clearml import Task
Expand All @@ -13,8 +12,6 @@
from spider.evaluation import build_foreign_key_map_from_json, evaluate
from spider.utils import extract_sql, is_valid_sql

nltk.download("punkt")

SEED = 42
DATASET_PATH = _ROOT / "data" / "spider"
PROMPT_TEMPLATE = """/* Given the following database schema: */
Expand Down

0 comments on commit a0c6087

Please sign in to comment.