Conversation
Signed-off-by: lvliang-intel <liang1.lv@intel.com>
|
Azure Pipelines: 6 pipeline(s) were filtered out due to trigger conditions. 1 pipeline(s) require an authorized user to comment /azp run to run. |
for more information, see https://pre-commit.ci
|
Azure Pipelines: 6 pipeline(s) were filtered out due to trigger conditions. 1 pipeline(s) require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
Pull request overview
Adds a set of Claude “skills” (workflow checklists/how-tos) under .claude/skills/ to standardize common contributor tasks in the AutoRound repo (PR review, adding data types, export formats, VLM support, and inference backends).
Changes:
- Introduces a PR review skill with an AutoRound-specific checklist (incl. quantization and CN translation parity checks).
- Adds step-by-step contributor skills for adding VLM models, quantization data types, export formats, and inference backends.
- Adds an index/readme for the available skills.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
.claude/skills/review-pr/SKILL.md |
PR review workflow/checklist tailored to AutoRound. |
.claude/skills/readme.md |
Overview/index of available skills. |
.claude/skills/add-vlm-model/SKILL.md |
Workflow for integrating new VLM support (blocks/templates/handlers/tests/docs). |
.claude/skills/add-quantization-datatype/SKILL.md |
Workflow for adding and registering a new quantization data type. |
.claude/skills/add-inference-backend/SKILL.md |
Workflow for registering and implementing a new inference backend. |
.claude/skills/add-export-format/SKILL.md |
Workflow for adding a new export format and wiring it into formats/backends. |
|
I was wondering whether it’s possible to leverage this to automatically generate export-format code for MAC |
The generated code may require debugging and is not guaranteed to function correctly out of the box. |
Signed-off-by: lvliang-intel <liang1.lv@intel.com>
…-round into lvl/add_claude_skills
|
Azure Pipelines: 6 pipeline(s) were filtered out due to trigger conditions. 1 pipeline(s) require an authorized user to comment /azp run to run. |
for more information, see https://pre-commit.ci
|
Azure Pipelines: 6 pipeline(s) were filtered out due to trigger conditions. 1 pipeline(s) require an authorized user to comment /azp run to run. |
|
Hi @lvliang-intel , how about we get this landed first and iterate with more updates afterward? |
Sure |
Signed-off-by: lvliang-intel <liang1.lv@intel.com>
…-round into lvl/add_claude_skills
|
/azp run Unit-Test-CUDA-AutoRound |
|
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
Description
Add Claude skills for AutoRound
Type of Change
Related Issues
Fixes or relates to #
Checklist Before Submitting