-
Notifications
You must be signed in to change notification settings - Fork 598
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(chat): Fold the output of the reference content of the search mo… #83
base: master
Are you sure you want to change the base?
Conversation
…del AND remove reference content from prompt
我的建议是联网的参考检索,也放进思考reasoning_content里,各个客户端现在也都对官方api做了兼容,肯定没啥问题 |
+1 感觉放在 reasoning_content里更合适 |
我不赞同把字段定义混淆的行为,这种行为可以明确是屎山代码的开始 并且那只是看上去会比较方便,使用起来并不理想,比如客户端或不支持思考内容 markdown 渲染导致页面过长,使用非思维链模型时出现思考内容从而造成困扰等等 既然 |
我们没法定义新的字段,因为不会有客户端支持,静默模式下放在reasoning_content里是最好的选择,普通模型可以全部输出来,但是静默模式,放在reasoning_content我认为是一种更好的选择, |
看上去静默不输出是现有功能,仔细阅读README,不过我暂时并没有使用静默模式的需求,不考虑对它进行修改,It's not my busniess at now
我使用的所有客户端都支持markdown,markdown语法也已经成为输出格式共识,包括众多模型官方web或app,或许你应该去提 feature request 给那些你了解的客户端 |
放到content里面在多次会话时会污染上下文,目前退而求其次放到reasoning_content里可能是当下最优的选择了 |
😩 或许你应该先 review 一下代码,就几行而已,又或许你至少要看一下 pull request 的描述。 |
抱歉是我表达的不清楚,在使用api的时候,多轮聊天过程中有时候会开个分支会话,然后使用其它模型回答进行比较,其它模型回答的时候是不会去掉检索内容的,所以在使用其它模型进行对比回答时会污染上下文,并且浪费token |
个人看法:无论用的什么插件,最后混在上下文里的如果只有网页标题和链接而没有实质内容的话,它带来的干扰要比它所含的信息要大的多 |
#64


使用 markdown 标签折叠联网的 参考检索,并且在多轮发送消息是将先去的 参考检索 去除掉,否则无法正常使用,会返回“还没学会”