Add None check for vLLM tokenizer

- Check _vllm_tok is not None before accessing attributes
- Use getattr for safer chat_template access
This commit is contained in:
Daniel Han 2026-01-05 10:02:11 +00:00
commit aff2dc9061

View file

@ -701,7 +701,7 @@ def _patch_trl_rl_trainers(trainer_file = "grpo_trainer"):
"if hasattr(self, 'llm') and self.llm is not None and hasattr(self.llm, 'get_tokenizer'):\n"
" _vllm_tok = self.llm.get_tokenizer()\n"
" _pc = getattr(self, 'processing_class', None) or getattr(self, 'tokenizer', None)\n"
" if _pc is not None and getattr(_pc, 'chat_template', None) is not None and _vllm_tok.chat_template is None:\n"
" if _vllm_tok is not None and _pc is not None and getattr(_pc, 'chat_template', None) is not None and getattr(_vllm_tok, 'chat_template', None) is None:\n"
" _vllm_tok.chat_template = _pc.chat_template\n"
"pass\n"
)