Skip to content

Conversation

@Sjshi763
Copy link
Contributor

@Sjshi763 Sjshi763 commented Nov 14, 2025

修改为使用轨迹流动嵌入模型

#62

Sourcery 总结

文档:

  • 将 PPIO 免费嵌入部分替换为新的 SiliconFlow BAAI/bge-m3 配置演练
Original summary in English

Summary by Sourcery

Documentation:

  • 将原先基于 PPIO 的免费向量模型使用指南,替换为在 AstrBot 中配置免费的 SiliconFlow BAAI/bge-m3 向量模型的分步教程,包括必填字段说明和示例截图。
Original summary in English

Summary by Sourcery

Documentation:

  • Replace the PPIO-based free embedding model guide with a step-by-step walkthrough for configuring the free SiliconFlow BAAI/bge-m3 embedding model in AstrBot, including required fields and example screenshots.

Pochinki98 and others added 2 commits October 18, 2025 19:27
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你好 - 我已审阅了你的更改,它们看起来很棒!

AI 代理提示
请处理此代码审查中的评论:

## 个别评论

### 评论 1
<location> `use/knowledge-base.md:132` </location>
<code_context>
+描述信息用于确认密钥,随便填
+然后复制密钥到`AstrBot`中
+![复制key](../source/images/knowledge-base/image-11.png)
+最后获取`API Base URL`,我直接给大家跳关填`https://api.siliconflow.cn/v1`
+这样我们就完成了嵌入模型的配置![成果](../source/images/knowledge-base/image-12.png)
\ No newline at end of file
</code_context>

<issue_to_address>
**suggestion:** Replace '跳关' with a more standard phrase.

"直接填写" or "直接提供" are clearer and more standard alternatives to "跳关".

```suggestion
最后获取`API Base URL`,我直接给大家提供`https://api.siliconflow.cn/v1`
```
</issue_to_address>

Sourcery 对开源项目免费 - 如果你喜欢我们的评论,请考虑分享它们 ✨
帮助我更有用!请点击每个评论上的 👍 或 👎,我将利用这些反馈来改进你的评论。
Original comment in English

Hey there - I've reviewed your changes and they look great!

Prompt for AI Agents
Please address the comments from this code review:

## Individual Comments

### Comment 1
<location> `use/knowledge-base.md:132` </location>
<code_context>
+描述信息用于确认密钥,随便填
+然后复制密钥到`AstrBot`中
+![复制key](../source/images/knowledge-base/image-11.png)
+最后获取`API Base URL`,我直接给大家跳关填`https://api.siliconflow.cn/v1`
+这样我们就完成了嵌入模型的配置![成果](../source/images/knowledge-base/image-12.png)
\ No newline at end of file
</code_context>

<issue_to_address>
**suggestion:** Replace '跳关' with a more standard phrase.

"直接填写" or "直接提供" are clearer and more standard alternatives to "跳关".

```suggestion
最后获取`API Base URL`,我直接给大家提供`https://api.siliconflow.cn/v1`
```
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New security issues found

@Sjshi763
Copy link
Contributor Author

炸了,开ide解决冲突

@Sjshi763 Sjshi763 marked this pull request as draft November 21, 2025 09:59
@Sjshi763 Sjshi763 marked this pull request as ready for review November 21, 2025 12:01
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New security issues found

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there - 我已经审阅了你的修改,这里是一些反馈:

  • 目前说明文档中把纯文本、行内截图以及带反引号的标签混在一起,整体阅读起来有点费力;建议把步骤重组为更清晰、统一编号的清单(例如分别为 API Key、API Base URL、Model ID 和 embedding 维度做小节),并在每张图片周围搭配更短、更聚焦的说明句。
  • 这句 截至2025/11/1 硅基流动的BAAI/bge-m3还是免费的 把一个具体日期直接写死在文档里,很快就会过时;建议改成不要依赖特定截止日期的表述(或者增加一句提示,让用户到硅基流动官网自行确认最新价格)。
给 AI 智能体使用的提示词(Prompt for AI Agents)
Please address the comments from this code review:

## Overall Comments
- The instructions mix plain text with inline screenshots and backticked labels in a way that’s a bit hard to follow; consider restructuring the steps into a clearer, consistently numbered checklist (e.g., separate sections for API Key, API Base URL, Model ID, and embedding dimension) with shorter sentences around each image.
- The line `截至2025/11/1 硅基流动的`BAAI/bge-m3`还是免费的` hardcodes a date that will quickly become stale; consider rephrasing to avoid a specific cutoff date (or add a note reminding users to verify current pricing on the SiliconFlow site).

## Individual Comments

### Comment 1
<location> `zh/use/knowledge-base.md:57` </location>
<code_context>
+[硅基流动官网](https://siliconflow.cn/)
+1. 注册账号
+2. ![实名认证](../source/images/knowledge-base/image-2.png)过一下实名
+3. 打开`Astrbot`,`模型提供商`,选择`新增模型提供商`
+![新增模型提供商](../source/images/knowledge-base/image-3.png)
+`嵌入(embedding)`,`OpenAI embedding`
</code_context>

<issue_to_address>
**suggestion (typo):** Unify the capitalization of “AstrBot” throughout the document.

Here it appears as `Astrbot`, while later you use `AstrBot` (e.g., in "复制密钥到`AstrBot`中"). Please standardize on a single capitalization, such as `AstrBot`, throughout.

Suggested implementation:

```
3. 打开`AstrBot`,`模型提供商`,选择`新增模型提供商`

```

在当前文件中应全局搜索所有变体(如 `Astrbot`, `astrBot`, `astrbot` 等),统一替换为 `AstrBot`,以确保整篇文档中的大小写一致。例如后文提到 “复制密钥到`AstrBot`中” 已经是正确形式,无需修改,但若还有其它写法不一致的地方,请一并统一为 `AstrBot`。
</issue_to_address>

### Comment 2
<location> `zh/use/knowledge-base.md:87` </location>
<code_context>
+描述信息用于确认密钥,随便填
+然后复制密钥到`AstrBot`中
+![复制key](../source/images/knowledge-base/image-11.png)
+最后获取`API Base URL`,我直接给大家跳关填`https://api.siliconflow.cn/v1`
+这样我们就完成了嵌入模型的配置![成果](../source/images/knowledge-base/image-12.png)
\ No newline at end of file
</code_context>

<issue_to_address>
**issue (typo):** “跳关填” looks like a typo; consider correcting the phrase.

This phrase is unclear and likely a typo. Consider rephrasing to something like “跳过讲解,直接填” or “我直接给大家,填 `https://api.siliconflow.cn/v1` 即可” for clarity.

```suggestion
最后获取`API Base URL`,我直接给大家填`https://api.siliconflow.cn/v1` 即可
```
</issue_to_address>

Sourcery 对开源项目免费使用——如果你觉得这些代码审查有帮助,欢迎分享给更多人 ✨
帮我变得更有用!请在每条评论上点一下 👍 或 👎,你的反馈将用于改进后续代码审查体验。
Original comment in English

Hey there - I've reviewed your changes - here's some feedback:

  • The instructions mix plain text with inline screenshots and backticked labels in a way that’s a bit hard to follow; consider restructuring the steps into a clearer, consistently numbered checklist (e.g., separate sections for API Key, API Base URL, Model ID, and embedding dimension) with shorter sentences around each image.
  • The line 截至2025/11/1 硅基流动的BAAI/bge-m3还是免费的 hardcodes a date that will quickly become stale; consider rephrasing to avoid a specific cutoff date (or add a note reminding users to verify current pricing on the SiliconFlow site).
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The instructions mix plain text with inline screenshots and backticked labels in a way that’s a bit hard to follow; consider restructuring the steps into a clearer, consistently numbered checklist (e.g., separate sections for API Key, API Base URL, Model ID, and embedding dimension) with shorter sentences around each image.
- The line `截至2025/11/1 硅基流动的`BAAI/bge-m3`还是免费的` hardcodes a date that will quickly become stale; consider rephrasing to avoid a specific cutoff date (or add a note reminding users to verify current pricing on the SiliconFlow site).

## Individual Comments

### Comment 1
<location> `zh/use/knowledge-base.md:57` </location>
<code_context>
+[硅基流动官网](https://siliconflow.cn/)
+1. 注册账号
+2. ![实名认证](../source/images/knowledge-base/image-2.png)过一下实名
+3. 打开`Astrbot`,`模型提供商`,选择`新增模型提供商`
+![新增模型提供商](../source/images/knowledge-base/image-3.png)
+`嵌入(embedding)`,`OpenAI embedding`
</code_context>

<issue_to_address>
**suggestion (typo):** Unify the capitalization of “AstrBot” throughout the document.

Here it appears as `Astrbot`, while later you use `AstrBot` (e.g., in "复制密钥到`AstrBot`中"). Please standardize on a single capitalization, such as `AstrBot`, throughout.

Suggested implementation:

```
3. 打开`AstrBot`,`模型提供商`,选择`新增模型提供商`

```

在当前文件中应全局搜索所有变体(如 `Astrbot`, `astrBot`, `astrbot` 等),统一替换为 `AstrBot`,以确保整篇文档中的大小写一致。例如后文提到 “复制密钥到`AstrBot`中” 已经是正确形式,无需修改,但若还有其它写法不一致的地方,请一并统一为 `AstrBot`。
</issue_to_address>

### Comment 2
<location> `zh/use/knowledge-base.md:87` </location>
<code_context>
+描述信息用于确认密钥,随便填
+然后复制密钥到`AstrBot`中
+![复制key](../source/images/knowledge-base/image-11.png)
+最后获取`API Base URL`,我直接给大家跳关填`https://api.siliconflow.cn/v1`
+这样我们就完成了嵌入模型的配置![成果](../source/images/knowledge-base/image-12.png)
\ No newline at end of file
</code_context>

<issue_to_address>
**issue (typo):** “跳关填” looks like a typo; consider correcting the phrase.

This phrase is unclear and likely a typo. Consider rephrasing to something like “跳过讲解,直接填” or “我直接给大家,填 `https://api.siliconflow.cn/v1` 即可” for clarity.

```suggestion
最后获取`API Base URL`,我直接给大家填`https://api.siliconflow.cn/v1` 即可
```
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

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