Skip to content

Migrate knowledge base charts from JSON to GPT-Vis syntax format#334

Merged
hustcc merged 7 commits intoaifrom
copilot/optimize-knowledge-base-markdown
Feb 3, 2026
Merged

Migrate knowledge base charts from JSON to GPT-Vis syntax format#334
hustcc merged 7 commits intoaifrom
copilot/optimize-knowledge-base-markdown

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

  • 1. Delete "数据文本 - Vis Text.md" knowledge base file
  • 2. Create new "总结摘要 - Summary.md" knowledge base file with proper naming format
  • 3. Update all chart knowledge base files to convert JSON examples to syntax format
    • All 26 chart types converted
  • 4. Ensure markdown file naming format is consistent across all knowledge base files
  • 5. Verify all changes are correct (tests passing, code review clean, security scan clean)
  • 6. Address feedback on Summary knowledge base:
    • Added missing entity types (dim_name, delta_value_pos/neg, ratio_value_pos/neg)
    • Converted entity types list to table format with examples
    • Removed vis prefix from all Summary examples
    • Enhanced T8 syntax reference with complete document structure (headings, paragraphs, lists, text formatting)
Original prompt

知识库 knowledges 中的 markdown 优化:

  1. 删除 vis text 知识库
  2. 增加 summary 知识库
  3. 所有图表知识库内容中的 json 配置改成 syntax 语法,并一定要确保正确性。
  4. 知识库 markdown 文件名格式风格保持一致

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@changeset-bot
Copy link

changeset-bot bot commented Feb 3, 2026

⚠️ No Changeset found

Latest commit: e9fcc10

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI and others added 2 commits February 3, 2026 00:37
…r charts to syntax

Co-authored-by: hustcc <7856674+hustcc@users.noreply.github.com>
…iles

Co-authored-by: hustcc <7856674+hustcc@users.noreply.github.com>
@hustcc
Copy link
Member

hustcc commented Feb 3, 2026

@copilot summary 有几个问题:

  1. 实体类型介绍不全,应该缺几个
  2. 每一个实体都要加一个简单的案例,最好用 table 来表述
  3. 只有 summary 的语法不需要加 vis 前缀

Copilot AI changed the title [WIP] Optimize markdown in knowledge base Migrate knowledge base charts from JSON to GPT-Vis syntax format Feb 3, 2026
Copilot AI requested a review from hustcc February 3, 2026 00:52
@hustcc
Copy link
Member

hustcc commented Feb 3, 2026

fixed #335

@hustcc
Copy link
Member

hustcc commented Feb 3, 2026

@copilot summary 关于 T8 语法的表述部分,可以参考 src/vis/summary/README.md

…table format, remove vis prefix

Co-authored-by: hustcc <7856674+hustcc@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 3, 2026

@copilot summary 有几个问题:

  1. 实体类型介绍不全,应该缺几个...

已完成修改(commit 0238285):

  1. ✅ 补充了缺失的实体类型:dim_namedelta_value_posdelta_value_negratio_value_posratio_value_neg
  2. ✅ 将实体类型改为 table 格式,每个实体都包含描述和示例
  3. ✅ 移除了所有 Summary 示例中的 vis 前缀

…ment structure details

Co-authored-by: hustcc <7856674+hustcc@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 3, 2026

@copilot summary 关于 T8 语法的表述部分,可以参考 src/vis/summary/README.md

已参考 src/vis/summary/README.md 完善 T8 语法表述(commit 5f77cab):

  • ✅ 添加文档结构说明(标题、段落、列表)
  • ✅ 添加文本格式说明(粗体、斜体、下划线、链接)
  • ✅ 完善实体标注语法说明,包含完整示例

@hustcc hustcc marked this pull request as ready for review February 3, 2026 01:14
@hustcc hustcc merged commit a341a4c into ai Feb 3, 2026
4 checks passed
@hustcc hustcc deleted the copilot/optimize-knowledge-base-markdown branch February 3, 2026 01:27
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