[A2-K]: 王群超 - flashattention - #142
Conversation
WillQvQ
left a comment
There was a problem hiding this comment.
按当前放宽后的标准复核(缺少飞书文档、内部路径/机器信息、Git 总结细节均不作为否决项),仍有以下阻塞问题需要修正:
-
正式矩阵的起始显存证据不可核验。
results/run_metadata.json写的是开跑前仅 5.58 GiB free;而flash_benchmark.py实际在全部 benchmark 循环结束后才调用run_metadata(),因此该值并非开跑前采集。README 又勾选了“开跑前不少于 22 GiB”,三者互相矛盾。请在空闲的单张 4090 上、首次 CUDA allocation 前采集并校验 free memory(至少 22 GiB),按独立进程重新生成正式矩阵及 metadata/memory evidence/报告。 -
results/correctness.json实际为 56 项pass=true、16 项pass=false,但 README 写成 72/72 passed。请修正实现或采用明确、合理且按 dtype 记录的容差后重新运行,确保 JSON 中包含可审核的 tolerance,报告与原始结果一致;不能把pass=false汇总成通过。 -
PR 标题应严格改为
[A2-K] 王群超 - flashattention(当前[A2-K]: ...多了冒号),并请把 PR 描述中的姓名、作业编号和 A2-K checklist 填写正确。
本次已确认:改动仅在本人 A2-K 目录,固定 CSV 组合完整,真实 Triton kernel、6/6 官方测试记录、附件大小和敏感信息检查均无其他阻塞问题。请直接在本 PR 推送修正,不需要另开 PR。
审核者:GPT-5.6 Sol Ultra
PR Checklist
基本信息
修改范围
students/_template、公共题面或仓库配置。[A编号] 姓名 - 简短说明。公开性与安全
git diff --cached,没有提交大型数据、模型权重、缓存或完整日志。双层提交
README.md提供可公开、已脱敏的作业报告。README.md已填写飞书补充文档链接。A1 额外检查
若本 PR 不是 A1,请将本节标记为不适用。
README.md、submission/cs336_basics/、submission/tests/adapters.py、submission/scripts/和logs/。README.md(Markdown)。assignment1-basics工作仓库位于 SummerQuest 的兄弟目录,且没有被提交到本 PR。../assignment1-basics运行官方测试,并使用同步脚本更新个人提交目录。A0 额外检查
若本 PR 不是 A0,请将本节标记为不适用。
nvidia-smi与gpustat并记录退出码。A2-P 额外检查
若本 PR 不是 A2-P,请将本节标记为不适用。A2-P 已正式发布,按题面要求提交。
README.md、submission/profiling/**/*.py、规定的轻量results/和至少三张被报告引用的裁剪、压缩图片。scripts/sync_a2p_submission.py和scripts/validate_repo.py。torch.profiler完成两个模型规模 × 三个 context 的六个train_steptrace。.nsys-rep、snapshot、完整 trace、权重、数据、压缩包、上游代码或依赖环境。results/与assets/合计不超过 2 MiB,Profile 只保留关键汇总与截图。A2-K 额外检查
若本 PR 不是 A2-K,请将本节标记为不适用。A2-K 已正式发布,按题面要求提交。
README.md、A2-K Python allowlist、全部规定的轻量results/和至少两张被报告引用的压缩图片。
scripts/sync_a2k_submission.py、官方 GPU tests 和scripts/validate_repo.py,且没有把 CUDA skip 写成 pass。前设置 23552 MiB allocator 上限。
memory_evidence.json已记录 allocator limit/fraction、peak allocated/reserved 和24 GiB 判定。
results/与assets/合计不超过 2 MiB,关键数字可回到轻量结果与命令。给助教的说明
<说明未完成项、环境限制或希望重点审核的内容。不要粘贴密钥和内部地址。>