Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
c99c130
fix EVERYTHING
ArthurZucker Aug 1, 2023
acf31e2
more fixes
ArthurZucker Aug 1, 2023
7305aff
⚗️⚗️ Tokenizer magic ⚗️⚗️
ArthurZucker Aug 1, 2023
01b8347
wrong value but test passes for the TODO
ArthurZucker Aug 1, 2023
b9ddbbb
update
ArthurZucker Aug 1, 2023
83af718
updat
ArthurZucker Aug 1, 2023
0babe38
safe protobuf import?
ArthurZucker Aug 1, 2023
0fdf51e
style
ArthurZucker Aug 1, 2023
2d197a1
non gated repo
ArthurZucker Aug 1, 2023
e9c7a72
update
ArthurZucker Aug 1, 2023
94964cd
fixup
ArthurZucker Aug 1, 2023
cc9ddcf
Merge branch 'main' of https://github.com/huggingface/transformers in…
ArthurZucker Aug 1, 2023
45cae43
Update src/transformers/models/llama/tokenization_llama.py
ArthurZucker Aug 2, 2023
53557a9
Update src/transformers/models/llama/tokenization_llama.py
ArthurZucker Aug 2, 2023
e049d11
Update tests/models/t5/test_tokenization_t5.py
ArthurZucker Aug 2, 2023
b64b2d2
nits
ArthurZucker Aug 2, 2023
cb95361
fix t5 too
ArthurZucker Aug 2, 2023
a86bf78
use assert equal
ArthurZucker Aug 2, 2023
913cd1d
fix llama decoding
ArthurZucker Aug 2, 2023
ef28574
nits on t5
ArthurZucker Aug 2, 2023
4f65261
fixup
ArthurZucker Aug 2, 2023
ad7f8c6
only remove the prefix space, not other spaces
ArthurZucker Aug 2, 2023
76d00cc
more deconding tests and more todos
ArthurZucker Aug 2, 2023
9cb92b6
fix CI as well
ArthurZucker Aug 2, 2023
204153f
fixup
ArthurZucker Aug 2, 2023
9f37103
skip failing test on CI (its tf its ok)
ArthurZucker Aug 2, 2023
700ee64
Merge branch 'main' of https://github.com/huggingface/transformers in…
ArthurZucker Aug 3, 2023
4b5315b
skip test_subword_regularization_tokenizer that is also crashing on t…
ArthurZucker Aug 3, 2023
a4ed16f
Merge branch 'main' of https://github.com/huggingface/transformers in…
ArthurZucker Aug 16, 2023
e7906c2
update llama
ArthurZucker Aug 17, 2023
ad33c97
revert good fixes
ArthurZucker Aug 17, 2023
f890882
fixup
ArthurZucker Aug 17, 2023
b7f98bc
empty
ArthurZucker Aug 17, 2023
bb79083
explain why we need to encode with an additional token
ArthurZucker Aug 17, 2023
3f8ac96
better warning?
ArthurZucker Aug 17, 2023
4249986
nits
ArthurZucker Aug 17, 2023
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
Prev Previous commit
Next Next commit
Update src/transformers/models/llama/tokenization_llama.py
Co-authored-by: amyeroberts <[email protected]>
  • Loading branch information
ArthurZucker and amyeroberts authored Aug 2, 2023
commit 45cae43a952e6ca3357670d5bf88d8fbd7bb5fd2
9 changes: 5 additions & 4 deletions src/transformers/models/llama/tokenization_llama.py
Original file line number Diff line number Diff line change
Expand Up @@ -209,10 +209,11 @@ def _tokenize(self, text):
the extra `SPIECE_UNDERLINE` prepended.
"""
if not self.legacy:
text = self.unk_token + text
tokens = self.sp_model.encode(text, out_type=str)
return tokens[self.unk_token_length :]
return self.sp_model.encode(text, out_type=str)
if self.legacy:
return self.sp_model.encode(text, out_type=str)
text = self.unk_token + text
tokens = self.sp_model.encode(text, out_type=str)
return tokens[self.unk_token_length :]

def _convert_token_to_id(self, token):
"""Converts a token (str) in an id using the vocab."""
Expand Down