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

fix: Translation error when passing empty user messages to certain models.(Revised) #1664

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

Konjac-XZ
Copy link
Contributor

@Konjac-XZ Konjac-XZ commented Feb 14, 2025

真是不好意思,刚刚晚上发版之后才发现那个 role: user 消息留一个空格的方法只对腾讯云的部署是一个有效的解决方案……测试不充分,红豆泥私密马赛。

这次应该是一个逻辑上“完全正确”的解决方法了,我想不出来哪个 API 必须有 role: system 不可,只有 role: user 是必要的。

Snipaste_2025-02-14_19-12-39
Snipaste_2025-02-14_19-11-46
Snipaste_2025-02-14_19-11-02

@Konjac-XZ Konjac-XZ changed the title fix: Translation error when passing empty user messages to certain models.(Refined) fix: Translation error when passing empty user messages to certain models.(Revised) Feb 14, 2025
@kangfenmao kangfenmao merged commit 26a5ae0 into CherryHQ:main Feb 14, 2025
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