Skip to content

Conversation

@zihenzzz
Copy link
Contributor

@zihenzzz zihenzzz commented Dec 22, 2025

Describe what this PR does / why we need it

Does this pull request fix one issue?

feat: #185

Describe how you did it

HTML报告本身就包含了图表代码(ECharts),通过iframe渲染后,浏览器自动执行JavaScript生成图表,所以用户在聊天框就能看到完整的带图表的报告.

Describe how to verify it

Special notes for reviews

@lzq986
Copy link
Contributor

lzq986 commented Dec 22, 2025

image 请验证功能完整再提交PR。

@lzq986
Copy link
Contributor

lzq986 commented Dec 22, 2025

你不要着急写代码提PR,你可以先梳理报告的图表是如何生成的,先看工作流是怎样远转的。

@zihenzzz
Copy link
Contributor Author

你不要着急写代码提PR,你可以先梳理报告的图表是如何生成的,先看工作流是怎样远转的。

image 我这边是可以的呀。。。 需求不是这样子的吗?

@lzq986
Copy link
Contributor

lzq986 commented Dec 22, 2025

你不要着急写代码提PR,你可以先梳理报告的图表是如何生成的,先看工作流是怎样远转的。

image 我这边是可以的呀。。。 需求不是这样子的吗?

图片从哪里来的,在哪里生成的,我看你的代码没有生成图片的逻辑。

@lzq986
Copy link
Contributor

lzq986 commented Dec 22, 2025

你不要着急写代码提PR,你可以先梳理报告的图表是如何生成的,先看工作流是怎样远转的。

image 我这边是可以的呀。。。 需求不是这样子的吗?

图片从哪里来的,在哪里生成的,我看你的代码没有生成图片的逻辑。
其实提这个需求有点那啥,因为现有的图表的生成是基于sql,数据分析结果 两者来生成的,比如销量预测,销量商品统计,它是根据sql结果和python代码执行结果,让LLM思考去画图比如折线图,饼图。 而不是生硬从报告抽出来的一个图。这个PR不用关闭,后面看怎么做我们再给你说吧。

@zihenzzz
Copy link
Contributor Author

你不要着急写代码提PR,你可以先梳理报告的图表是如何生成的,先看工作流是怎样远转的。

image 我这边是可以的呀。。。 需求不是这样子的吗?

图片从哪里来的,在哪里生成的,我看你的代码没有生成图片的逻辑。
其实提这个需求有点那啥,因为现有的图表的生成是基于sql,数据分析结果 两者来生成的,比如销量预测,销量商品统计,它是根据sql结果和python代码执行结果,让LLM思考去画图比如折线图,饼图。 而不是生硬从报告抽出来的一个图。这个PR不用关闭,后面看怎么做我们再给你说吧。

流式的生成这个图吗?(过程展示出来) 外面展示的和下载的那个图不一样?

@VLSMB VLSMB marked this pull request as draft December 24, 2025 08:46
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