Skip to content

fix-step3-readme#286

Merged
yaozhewei merged 3 commits into
deepspeedai:masterfrom
zhangfanTJU:fix-step3-readme
Apr 28, 2023
Merged

fix-step3-readme#286
yaozhewei merged 3 commits into
deepspeedai:masterfrom
zhangfanTJU:fix-step3-readme

Conversation

@zhangfanTJU
Copy link
Copy Markdown
Contributor

fix readme in step3.

```bash
applications/DeepSpeed-Chat/training/utils/model/model_utils.py#L60
```
⚡⚡⚡ When you use above script, please make sure you set `rlhf_training` is False
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch @zhangfanTJU. Could you add more context for other users to understanding where they should set rlhf_training to be false

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much for your reply and suggestions. I have revised the readme. Additionally, I did not provide a specific line number as the code is constantly being updated.

@yaozhewei yaozhewei merged commit a3e4857 into deepspeedai:master Apr 28, 2023
Syulin7 pushed a commit to Syulin7/DeepSpeedExamples that referenced this pull request May 15, 2023
* fix-step3-readme

* Update README.md

---------

Co-authored-by: Zhewei Yao <zheweiy@berkeley.edu>
leocnj pushed a commit to leocnj/DeepSpeedExamples that referenced this pull request May 27, 2023
* fix-step3-readme

* Update README.md

---------

Co-authored-by: Zhewei Yao <zheweiy@berkeley.edu>
hwchen2017 pushed a commit that referenced this pull request Jun 8, 2025
* fix-step3-readme

* Update README.md

---------

Co-authored-by: Zhewei Yao <zheweiy@berkeley.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants