Skip to content

Commit

Permalink
Don't print that we're gonna store LM preds for non-fast tokenizers
Browse files Browse the repository at this point in the history
  • Loading branch information
norabelrose committed Apr 17, 2023
1 parent fcf0c5f commit bbbee75
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion elk/extraction/extraction.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ def extract_hiddens(
cfg.model, truncation_side="left", verbose=False
)
has_lm_preds = is_autoregressive(model.config)
if has_lm_preds and rank == 0:
if has_lm_preds and rank == 0 and tokenizer.is_fast:
print("Model has language model head, will store predictions.")

# Iterating over questions
Expand Down

0 comments on commit bbbee75

Please sign in to comment.