Merged
docs: 明确知识仓与技能仓的内容边界 #134
docs: 明确知识仓与技能仓的内容边界 #134
Merged
汪超Add on 18 days ago
汪超Collaborators
18 days ago

Pull Request 说明

变更说明

  • 关联 Issue:
  • 主要改动:新增了文档用于描述知识库仓和skills仓的内容边界
  • 补充说明:

变更类型

验证

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for 汪超's contribution.)
汪汪超Collaborators
18 days ago Created pull request, commit 310eedb8
CANN-robotCANN-robotCollaborators
18 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
18 days ago Comments:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/cannbot-knowledge ✅ zhanghz1, wangsj0012 (2/2) ✅ zhanghz1 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

qq_41667743, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
汪超Collaborators
18 days ago Comments:

/compile

likedislike
汪汪超Collaborators
18 days ago Pre merge succeeded (commit_id: 4fa6b157273b585a91f602be8995bb8821a8783a)
CANN-robot
CANN-robotCollaborators
18 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
18 days ago Added label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
18 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
18 days ago Added label: ci-pipeline-passed
jiangZD17 days agoStart code review2
README.md
@@ -106,2 +106,4 @@
106106设计取舍见[设计原则](docs/design_principles.md);常见任务见[形成可追溯回答](recipes/answer_with_evidence.md)、[贡献知识](recipes/contribute_knowledge.md)、[查询 API](recipes/api_search.md)和[使用知识库进行算子开发](recipes/operator_development.md)。
107107 
108+哪些内容放技能仓、哪些内容放知识库仓,见 [cannbot-knowledge 与 cannbot-skills 的内容边界](docs/knowledge_and_skills_boundary.md)。
109+ 
jiangZD17 days agoComments:

建议不提cannbot-skill,只提knowledge仓的纳入范围,可对接到用户个人的skill。

likedislike
汪超Collaborators
17 days ago Comments:

已修改

汪汪超Collaborators
17 days ago Pre merge succeeded (commit_id: 1ffe08bcaf645a1229b6b7bf46f64b821e139140)
5 event messages are collapsed here。 See more
CANN-robotCANN-robotCollaborators
17 days ago Removed label: ci-pipeline-passed
CANN-robot
CANN-robotCollaborators
17 days ago Comments:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
汪超Collaborators
17 days ago Comments:

/compile

likedislike
汪汪超Collaborators
17 days ago Pre merge succeeded (commit_id: 6e38847729dc09efcffd6731cc798d38aaf84e23)
CANN-robot
CANN-robotCollaborators
17 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
17 days ago Added label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
17 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
17 days ago Added label: ci-pipeline-passed
JjiangZD
17 days ago Resolved final issue
zhanghz1
zhanghz1Collaborators
17 days ago Comments:

/approve

likedislike
CANN-robotCANN-robotCollaborators
17 days ago Added label: approved
wangshaojun
wangshaojunCollaborators
17 days ago Comments:

/lgtm

likedislike
CANN-robotCANN-robotCollaborators
17 days ago Added label: lgtm
CANN-robotCANN-robotCollaborators
17 days ago Merged pull request