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

[Bug] Attached Messages Count 无效 #2111

Closed
3 tasks done
songxiaocheng opened this issue Jun 23, 2023 · 0 comments
Closed
3 tasks done

[Bug] Attached Messages Count 无效 #2111

songxiaocheng opened this issue Jun 23, 2023 · 0 comments

Comments

@songxiaocheng
Copy link
Contributor

songxiaocheng commented Jun 23, 2023

为了提高交流效率,我们设立了官方 QQ 群和 QQ 频道,如果你在使用或者搭建过程中遇到了任何问题,请先第一时间加群或者频道咨询解决,除非是可以稳定复现的 Bug 或者较为有创意的功能建议,否则请不要随意往 Issue 区发送低质无意义帖子。

点击加入官方群聊

反馈须知

⚠️ 注意:不遵循此模板的任何帖子都会被立即关闭,如果没有提供下方的信息,我们无法定位你的问题。

请在下方中括号内输入 x 来表示你已经知晓相关内容。

  • 我确认已经在 常见问题 中搜索了此次反馈的问题,没有找到解答;
  • 我确认已经在 Issues 列表(包括已经 Close 的)中搜索了此次反馈的问题,没有找到解答。
  • 我确认已经在 Vercel 使用教程 中搜索了此次反馈的问题,没有找到解答。

描述问题
Attached Messages Count 设置无效。在我有限的测试中,无论值设置为多少,每次都会带上全部messages。

如何复现

  1. 全新docker部署,打开网页,设置Attached Messages Count为0。从浏览器开发者工具的Local Storage里面的模型配置中,historyMessageCount确实为0。
  2. 发送消息,随便聊,打开浏览器开发者工具,可以看到 openai payload 里面的messages是越来越多的。
  3. 多次询问同一个问题,其后续回答为:“I apologize for any confusion earlier. As I mentioned before,...”。

截图

Settings Local Storage DevTools

一些必要的信息

  • 系统:linux
  • 浏览器: chrome
  • 版本:20230620(docker), 20230623(本地)

其它
似乎和这里有关
https://github.com/Yidadaa/ChatGPT-Next-Web/blob/c636993989cb48a76b36bdf7d208ce6d34fce717/app/store/chat.ts#L386-L411
longTermMemoryStartIndex 始终为0,因为session.lastSummarizeIndex始终为0

@songxiaocheng songxiaocheng changed the title [Bug] Attached Message Count 无效 [Bug] Attached Messages Count 无效 Jun 23, 2023
Yidadaa added a commit that referenced this issue Jun 24, 2023
fix: #2111 make "Attached Messages Count" working
alchemist139 pushed a commit to alchemist139/ChatGPT-Next-Web that referenced this issue Sep 21, 2023
alchemist139 pushed a commit to alchemist139/ChatGPT-Next-Web that referenced this issue Sep 21, 2023
…message-count

fix: ChatGPTNextWeb#2111 make "Attached Messages Count" working
chenzeyu pushed a commit to neutronsg/ChatGPT-Next-Web that referenced this issue Nov 8, 2023
gaogao1030 pushed a commit to gaogao1030/ChatGPT-Next-Web that referenced this issue May 16, 2024
gaogao1030 pushed a commit to gaogao1030/ChatGPT-Next-Web that referenced this issue May 16, 2024
…message-count

fix: ChatGPTNextWeb#2111 make "Attached Messages Count" working
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

No branches or pull requests

1 participant