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

删除deserializer中是否为json的判断 #29

Closed
wants to merge 1 commit into from

Conversation

I-am-DJ
Copy link

@I-am-DJ I-am-DJ commented May 23, 2024

删除deserializer中是否为json的判断
openaiservice中function call对象改为了局部变量,qwen流式返回function call时,最后一个会带有finishReason但是没有具体的function call结果,导致重复进入解析,所以改为局部变量进行判断

openaiservice中function call对象改为了局部变量,qwen流式返回function call时,最后一个会带有finishReason但是没有具体的function call结果,导致重复进入解析,所以改为局部变量进行判断
Copy link
Owner

@Lambdua Lambdua left a comment

Choose a reason for hiding this comment

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

  • deserializer中的json判断不能删除,该反序列方法,不仅仅适用于stream接口. 非stream接口也是走的这个.所以需要判断,不过这边的json校验方法是需要优化一下,不能每次创建mapper实例
  • openaiService修改部分,是否有跑通过单元测试,我这边跑不通

@Lambdua Lambdua closed this May 27, 2024
@I-am-DJ
Copy link
Author

I-am-DJ commented May 28, 2024

这两天生病了,能方便加下其他联系方式吗?我尝试看下用chatgpt的api能否访问

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.

None yet

2 participants