Pull Request已成功合入, 合并人@CANN-robot
(感谢 wangshaojun 的贡献)Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-cann/cannbot-knowledge | ✅ qq_41667743, zhanghz1 (2/2) | ✅ zhanghz1 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
wangsj0012, thanks for your pull request. All authors of the commits have signed the CLA. 👍


这张知识卡的更新把多个不同根因、不同处置方式的案例合进了一张既有卡,已经违反本 PR 摄入规则要求的“一卡一根因”。例如:
- R2:ROIAlign 的误差比较问题;
- R17:GridSampler 因正负输入相消产生近零结果,作者建议把输入从 [-1,1] 改为 [0,1];
- R19:示例代码中的比较器修复;
- R20:C++ ST 比较器的 epsilon 取值问题。
最明显的是正文中编号为 R17 的案例:它通过修改 GridSampler 输入分布来规避相消,属于测试数据侧处置,与本卡标题对应的 “相对误差分母改用 max(|g|, th)”不是同一根因。当前用 GridSampler x 分布 0 1 相消 查询,前 50 条都找不到这条知识。
这个知识卡对应的多个补充案例信息需要重新审视,按实际情况拆成多个知识卡,不要把聚合卡以扩写既有卡的方式直接落库。


changed this line on 118b96c9 view diff detail
这段 ToFloat
PR 新增的另一张卡已经准确说明同一个错误:floattohalf_subnormal_threshold_exp102.md:192 明确指出规格化后的指数应为 127 - 14 - e,写成 127 - 15 - e 会使结果小一倍。因此当前 PR 内部存在直接矛盾。
agent建议最小修复是把 388 行的exp初始化为0,需要复核确认下


| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |


这段话把最大次正规数 0x03ff 写成“≈2^-15×(1−2^-10)”并说规格化后约为 2^-15。实际值是 (1023/1024)×2^-14 ≈ 6.09755516e-5,紧邻最小正规数 2^-14;现有文字仍少一倍。
AI建议修改:
次正规分支修正说明:这里的 exp 实际记录尾数的左移次数,须从 0 开始,每左移一次加 1;FP32 指数字段为 127 - 14 - exp(与代码中的 127 - 15 - exp + 1 等价)。原先从 1 开始,会使所有非零 FP16 次正规数的转换结果缩小一半。边界编码可在 host 侧按位型复算:0x0001(最小次正规)= 2^-24,0x0200 = 2^-15,0x03ff(最大次正规)= (1023/1024) × 2^-14 ≈ 6.09755516 × 10^-5;0x0400(最小正规)= 2^-14。这与 FloatToHalf 次正规阈值卡 (../precision/ floattohalf_subnormal_threshold_exp102.md) 中读方向使用的 127 - 14 - e 基指数一致。
可以再检查下对0x03ff的解读是不是搞错了


changed this line on 06241ebc view diff detail
新卡链接标题写成了 [0,1],而索引生成规则要求写成 \[0,1\]。未转义的方括号使同步工具无法识别这条已有链接;直接运行写入模式的 sync-indexes 会在末尾再追加一条指向同一卡片的链接。请只把第 380 行链接标题中的 [0,1] 改为 \[0,1\],再运行 python3 .agents/skills/knowledge-ingest/scripts/knowledge_ingest.py sync-indexes --check 确认输出 stale 0。


changed this line on 06241ebc view diff detail
/compile


| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |


/approve


The MR can not be merged, because of CodeReview discussion not resolved
If you want to solve this problem, you can click here to do it in the FAQs.


Pull Request 说明
变更说明
ops-knowledge-trace-ingest流水线(物化 → 归一化 → 相关性门禁 → 分片挖掘 → 语义聚类策展 → 多轮独立审查 → receipt 绑定原子发布)全量落库为 runbooks 实战知识:新增 578 张卡(compilation 300 / precision 109 / validations 104 / runtime 44 / performance 21),对 188 张存量卡做库内增量合并,共 766 个发布目标,并同步更新 Skill 参考资料与脚本。## 遗留问题节并入## 失效边界(以 未决项 条目承载原有语义),对齐 evals 既有回归(HEAD 存量语料 0 命中);trace-ingest Skill 5 份参考文档与materialize_trace_contract.py(augment_ledger 支持多 trace 簇源数组合并与代表草稿择优)同步更新。index.md与 3 份日志;.gitignore增加local-tools/(会话期临时基建,不入库)。raw_source_sha256去重精炼 + 本仓「遗留问题→失效边界」迁移),当前无冲突,可直接合入。变更类型
验证