Skip to content

feat: 图片显示, 用于异步MCP的图片下发并显示 #1154

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 8 commits into from
Closed

feat: 图片显示, 用于异步MCP的图片下发并显示 #1154

wants to merge 8 commits into from

Conversation

Ky1eYang
Copy link
Contributor

@Ky1eYang Ky1eYang commented Sep 3, 2025

  1. application.cc 添加OnMessage接收type="image"的情况, 包含url, width, height, format(jpeg, png, gif...)的图片信息
  2. application.cc 添加DisplayImage, 当接受到对应message时调用http然后调用SetPreviewImage显示图片
  3. （非Wexin聊天风格）更改lcd_display.cc的SetupUI，让PreViewImage上层是PreViewImage_Popup，然后添加了一个定时器的callback用的函数，要显示emo时调用，缺省5秒后给popup设置Hidden

现在有几个问题

  1. OnMessage的处理，image类型时是否要放进custom内，如果要，因为后续新增代码量较大，需要为custom类型写一个单独的较厚的json解析器吗？
  2. DisplayImage的代码量也比较多，而且现在还没有处理png/gif等其他情况，是否需要类似Audio一样做个ImageServer，代码也往里面归？

@78
Copy link
Owner

78 commented Sep 4, 2025

是因为你用了这个jpeg dec导致吃了很多SRAM出现内存不足的情况，SRAM过低会导致硬件不稳定以及开发者无法在此基础上扩展功能

@Ky1eYang
Copy link
Contributor Author

Ky1eYang commented Sep 5, 2025

是因为你用了这个jpeg dec导致吃了很多SRAM出现内存不足的情况，SRAM过低会导致硬件不稳定以及开发者无法在此基础上扩展功能

试了下这确实应该是这个库消耗了栈，我试了下把handler分到堆上也没能减少占用...

@78
Copy link
Owner

78 commented Sep 12, 2025

后续使用MCP实现，关闭此PR

@78 78 closed this Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ky1eYang @78