huggingface / huggingface/course
max_seq_length is no longer a valid param in SFTTrainer
Open
- Dominant language
- MDX
- Stars
- 4.2k
- Forks
- 1.4k
- Avg merge
- 13m
- Merged PRs (30d)
- 1
Description
Here is the PR for this fix: https://github.com/huggingface/course/pull/1170
The issue:
- max_seq_length is no longer a valid param in SFTTrainer, instead we use max_length in SFTConfig
- since this param is not important in this chapter, I delete the outdated line to avoid confusion
- Link: https://huggingface.co/docs/trl/main/en/sft_trainer
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.