Repository navigation
Conversation
🤖 AI Code Review
Full per-file results and batch artifacts Files without a review
|
watney1024
approved these changes
Oct 23, 2026
Preserve primary errors and signal exit codes when report persistence fails. Verify real Docker SIGINT/SIGTERM cleanup and executable reuse on 1/4-hart small graphs. Consolidate W5 development documentation and reproduction instructions.
Use cores consistently while retaining legacy CLI, Python and metadata aliases. Cover argparse attributes, dataclass serialization/replacement and conflicting configuration values without changing default single-core execution. Publish the expanded W5 generation/report explanations and historical summary. Separate reused builds, program elapsed time and container compilation time. Validation: 591 relevant W4/W5 interface and small-QEMU tests, plus 40 instruction-counter tests after merging main. No full model or Docker rerun.
Organize the W5 report around changes, measured results and reproduction. Keep timing and accuracy explanations below the results table, and move Linux/container preparation into the shared reproduction guide.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
W5 将 W4 的单次模型前向接成文本生成:本机 Python 处理提示词、逐步选 token 并校验,QEMU 中的 RISC-V 程序执行模型。三条冻结用例复用同一 ELF 和权重文件,支持命令行、多轮历史管理和干净容器部署。
阅读入口
链接均固定到本次提交 e72cd2d,不依赖本机 output 页面。
本次文档整理
原报告把结果、详细解释和环境准备交叉放置,读者需要来回寻找。本次仅修改两份 Markdown:
本轮未改变运行代码、数值策略或 CI 配置,也未重跑模型。
W5 功能与默认行为
--cores/cores,兼容--harts/harts、parse_args().harts、旧构建记录和 C 宏;asdict()导出一致的双字段,replace()更新任一名称时同步别名。已有完整模型结果
以下来自 2026-10-23、基线 8ad3474,不是本次提交的新测量:完整 28 层 Qwen3-0.6B、FP32、固定长度 256,显式 4 core / blocked_fma / 2×4 / column-panel / Mean two-four-lane-last-axis / nonlinear fp32-reference,padding 为 none。
3/3 用例、24/24 步通过。 每条生成 8 个新 token,无 KV cache,每步都执行完整前向并启动新 QEMU,没有重复编译。全部输出有限且严格
max_abs < 0.001,逐步 token、最终文字与停止原因和冻结参考一致。三条均因
max_new_tokens结束,不能视为自然回答完毕;具体续写见 实际输出表。端到端生成与校验时间包含最终文字生成及核对,不含前置编译、最终报告保存或终端显示。QEMU 时间已包含在端到端时间中,不能相加。干净容器单次前向和本机单次前向都生成
Paris,与 ORT 最大误差约 0.000172138,两边完整输出逐字节相同。本机内部计时 444.416 秒且复用构建;容器 547.454 秒,其中本次编译 93.868 秒。构建条件不同,不作为容器性能对比,也不代表容器三例完整生成已通过。验证与普通 CI
**本轮文档检查:**现有 Linux 文档检查器通过(129 份文档、245 个 shell 代码块),两份修改文档的 84 个本地链接及相关锚点通过;折叠结构、四项说明的位置、预览一致性及 diff 格式检查通过。没有为文档修改重复执行模型或代码测试。
**既有代码验证:**提交
3782bf8对应的相关验证为 631 passed,0 failed / error / skipped(18 项未选择)。真实 1/4 core 小图完整输出与 ORT 相同、最大误差为 0,超时/取消后的清理与复用通过;这不是本次文档提交重新运行的结果。main013b3c7已由此前合并提交整合。当前文档提交
e72cd2d的普通 CI #41068685444已完成,结果为 SUCCESS:14 个 job 成功、7 个按条件跳过,0 失败。以上是本次提交的远端 CI 结果;README“其他”中的旧 CI/benchmark 记录仍保留其历史版本标签。
未手动追加、请求或重跑 AI 审查;仓库随推送自动触发的审查按现有规则执行。
剩余范围
pull_request_target使用。本次不修改 WSL 配置,不合并 PR。