Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PPO trainer 能否支持下 args.max_steps #1325

Closed
liu-zichen opened this issue Oct 31, 2023 · 1 comment
Closed

PPO trainer 能否支持下 args.max_steps #1325

liu-zichen opened this issue Oct 31, 2023 · 1 comment
Labels
solved This problem has been already solved

Comments

@liu-zichen
Copy link
Contributor

liu-zichen commented Oct 31, 2023 •
edited
Loading

transformers Trainer中 max_steps 的优先级高于 num_train_epochs。但是目前 ppo_train 函数并未支持 max_steps 参数。

感谢🙏
@hiyouga hiyouga added enhancement New feature or request pending This problem is yet to be addressed labels Oct 31, 2023
@hiyouga
Copy link
Owner

hiyouga commented Nov 1, 2023

已修复

@hiyouga hiyouga closed this as completed in 083787d Nov 1, 2023
@hiyouga hiyouga added solved This problem has been already solved and removed enhancement New feature or request pending This problem is yet to be addressed labels Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
solved This problem has been already solved
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hiyouga @liu-zichen