Skip to content

[feature]支持ComfyUI的quick start文档测试看护 - #167

Open
yzeyu71 wants to merge 1 commit into
Ascend:mainfrom
yzeyu71:comfyui-quick-start-guard
Open

yzeyu71 wants to merge 1 commit into
Ascend:mainfrom
yzeyu71:comfyui-quick-start-guard

Conversation

@yzeyu71

@yzeyu71 yzeyu71 commented Sep 10, 2026

Copy link
Copy Markdown

概述

将 ComfyUI 的 quick start 看护从 workflows 仓库迁移到 docs 仓库,对齐 #142(peft)确立的模式:以 Markdown 文档作为测试用例(#test / #test-setup / #test-result 标注),由本仓库公共引擎 .github/workflows/quick-start-template.yml 在自托管 NPU runner 上执行并持续看护。

变更内容

文件 说明
sources/comfyui/index.md ComfyUI 文档入口,include 快速上手正文
sources/comfyui/quick_start.md 快速上手正文:环境检查 → 克隆 ComfyUI → 安装依赖 → 下载 SD 1.5 checkpoint → 启动 NPU 服务 → API 例程文生图 → PNG 结果校验
tests/comfyui/test_quick_start_ascend.py MarkdownDocTestBase 测试子类:CANN 环境注入、CUDA 排除表、torch 栈探针、HF 缓存状态上报与坏片清理
tests/comfyui/__init__.py 测试包引导(sys.path 注入)
.github/workflows/comfyui-quick-start.yml 项目薄触发器:12h 定时 + workflow_dispatch + PR 路径过滤(sources/comfyui/**tests/comfyui/**),复用公共 quick-start 引擎
index.rst 多模态分区注册 ComfyUI 卡片与 toctree 条目(置于 whisper_cpp 之后);已有内容零改动,仅两处纯新增

与 workflows 侧版本的差异

  • 模型下载由 ModelScope 改为 Hugging Face Hubstable-diffusion-v1-5/stable-diffusion-v1-5v1-5-pruned-emaonly.safetensors),依赖安装同步由 modelscope 替换为 huggingface_hub,符合 docs CI 统一走 HF 的约定。
  • 测试侧缓存校验对应切换为 HF 系函数(report_huggingface_state / purge_huggingface_corrupt / resolve_huggingface_cache)。

测试计划

  • 手动 workflow_dispatch 触发 comfyui-quick-start,确认自托管 NPU runner 上端到端出图成功
  • 确认 PR 路径过滤只在本目录变更时触发看护
  • 确认 index.rst 渲染后卡片与侧边栏导航正常

@ascend-robot

Copy link
Copy Markdown

CLA Signature Guide

@yzeyu71 , thanks for your pull request.

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

Commit Reason
[d43790a [feature]支持ComfyUI的quick ...](d43790a) the email used in the commit is not linked to a signed CLA!
please verify that it matches the email you used when signing the CLA.

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.

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.

2 participants