Partial fix for transformers 5.13.0

#11

This pr addresses broken tokenization config in transformers, a variable with special tokens was renamed from additional_special_tokens to extra_special_tokens. This was supposed to happen in v5, but the code path was hidden untill changes in this pr: https://github.com/huggingface/transformers/pull/46667/changes#diff-d804e851851cdebeb8048938f1f8beec1cfa78bf7b1f06af86faa450f9d18defR818-R823

This pr (the one I'm proposing here, on hugginface) does not change the behaviour for ealier versions of transformers, while fixing it in newest one. There is another problem related to this model, which I'm trying to fix in this pr: https://github.com/vllm-project/vllm/pull/47438.

Since the transformers error occures after the vllm one, in order to reproduce it you'll have to checkout to my vllm branch, and run:

vllm serve xlangai/OpenCUA-7B --dtype bfloat16 --tensor-parallel-size 1 --max-model-len 8192 --gpu-memory-utilization 0.85 -cc '{"inductor_compile_config":{"benchmark_combo_kernel":false}}' --port 8000 --trust-remote-code --limit-mm-per-prompt '{"image": 1}'
microslaw changed pull request status to closed

Sign up or log in to comment