Skip to content

[feature]支持ms-swift的quick start文档测试看护 - #141

Closed
Devin-DC-Huang wants to merge 2 commits into
Ascend:mainfrom
Devin-DC-Huang:main
Closed

[feature]支持ms-swift的quick start文档测试看护#141
Devin-DC-Huang wants to merge 2 commits into
Ascend:mainfrom
Devin-DC-Huang:main

Conversation

@Devin-DC-Huang

Copy link
Copy Markdown
Contributor
  1. 实现一个公共的github workflow模板支持quick start文档测试看护
  2. 实现ms-swift的quick start文档测试看护

1. 实现一个公共的github workflow模板支持quick start文档测试看护
2. 实现ms-swift的quick start文档测试看护
@ascend-robot

Copy link
Copy Markdown

CLA Signature Guide

@Devin-DC-Huang , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit Reason
[93afcb2 [feature]支持ms-swift的quick...](93afcb2) the email used in the commit is an invalid email!
please check all commits.

To sign CLA, click here.

To check if your email is configured correctly, refer to the FAQs.

Once you've signed the CLA or updating your email, please comment /check-cla to revalidate CLA status.

========

NPU 相关文档由 ms-swift 官方维护,请访问官方文档站:
ms-swift 通用文档由 ms-swift 官方维护。本页收录 Ascend NPU 适配的快速上手。

@zheliuyu zheliuyu Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

感谢你的贡献。

就像这里的注释所说,以及以下 RFC 和 PR 所讨论的那样:如果 ms-swift 仓中提供了环境安装指导以及 Ascend NPU 的快速上手文档,我们应该复用它们,而不是在 Ascend/docs 中额外维护一份。

如果您觉得 ms-swift 的快速上手文档不足以指导用户使用,建议直接将此文档内贡献在 ms-swift 项目中,Ascend/docs 会同步更新。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

感谢你的答复。

后续也会往ms-swift社区合并关于ascend npu的快速开始的文档。当前ms-swift社区只有一份cuda相关的快速开始文档(https://swift.readthedocs.io/zh-cn/latest/GetStarted/Quick-start.html),npu只是简单描述了是支持的硬件(https://swift.readthedocs.io/zh-cn/latest/GetStarted/SWIFT-installation.html)。

计划是现在ascend doc这个仓库合并ms-swift社区暂时没有npu的快速开始文档,后续考虑往ms-swift贡献对应的文档。

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

NPU 的环境安装和快速开始文档在这里:https://swift.readthedocs.io/zh-cn/latest/BestPractices/NPU-support.html
在 Ascend/docs,即本项目中,我也添加了它 https://github.com/Ascend/docs/blob/main/index.rst#ms-swift

所以这里并不建议在额外添加一份 ms-swift 的快速开始文档。

如果仅是想为 Ascend/docs 增加文档测试看护的能力,可以选择其他项目。

@ascend-robot

Copy link
Copy Markdown

CLA Signature Guide

@Devin-DC-Huang , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit Reason
[93afcb2 [feature]支持ms-swift的quick...](93afcb2) the email used in the commit is an invalid email!
please check all commits.

To sign CLA, click here.

To check if your email is configured correctly, refer to the FAQs.

Once you've signed the CLA or updating your email, please comment /check-cla to revalidate CLA status.

github-actions Bot pushed a commit that referenced this pull request Sep 1, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

📖 文档预览已随本 PR 关闭而清理。

pr中涉及quick start的文档或者测试代码变化,触发ci
@ascend-robot

Copy link
Copy Markdown

CLA Signature Guide

@Devin-DC-Huang , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit Reason
[93afcb2 [feature]支持ms-swift的quick...](93afcb2) the email used in the commit is an invalid email!
please check all commits.

To sign CLA, click here.

To check if your email is configured correctly, refer to the FAQs.

Once you've signed the CLA or updating your email, please comment /check-cla to revalidate CLA status.

github-actions Bot pushed a commit that referenced this pull request Sep 1, 2026
github-actions Bot added a commit that referenced this pull request Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants