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.

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

feat: add finetuning dependency packages #526

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

leiqichn
Copy link

非常感谢组织提供的Demo, 给我提供了很好的指导和学习。但是我在实际微调过程中发现我们finetune_demo/requirements.txt缺少了两个依赖包。特此提个PR,贡献自己的一份力量,帮助后边开发者。谢谢。

@zRzRzRzRzRzRzR
Copy link
Member

zRzRzRzRzRzRzR commented Aug 31, 2024

按照readme所说和req要求,应该先安装推理的,也就是主页的依赖,如果你正常安装了推理的req后再安装微调的,应改是不会出现这个问题的呀

@leiqichn
Copy link
Author

leiqichn commented Sep 1, 2024

按照readme所说和req要求,应该先安装推理的,也就是主页的依赖,如果你正常安装了推理的req后再安装微调的,应改是不会出现这个问题的呀

建议解耦比较好一点吧?有人想直接推理,有人想直接微调的。

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