fix: prefer `| trim` over `.strip()`in tokenizer_config
#21
by
drbh
HF staff
- opened
this PR simply replaces .strip()
with trim
to align with standard jinja: https://jinja.palletsprojects.com/en/2.11.x/templates/#list-of-builtin-filters
Thank you! Tested locally, works for me. cc @Rocketknight1 in case he sees some incompatibility.
@huggingface/jinja
v0.1.3 is now out which adds support for | trim
(link). Feel free to merge :)
Thanks all, merging!
pcuenq
changed pull request status to
merged