[A2-K] 李畅松 - 提交 attention 实验结果 - #158
Open
maybe1possible wants to merge 3 commits into
Open
Conversation
WillQvQ
requested changes
Aug 8, 2026
WillQvQ
left a comment
Contributor
There was a problem hiding this comment.
审核结论:需要修改后再合并。
实现、108 项扩展正确性、6 项 CUDA tests、checkpoint/attention/compile/Flash 矩阵、附件与范围均已核验;当前有两项阻塞:
- 提交代码中没有任何
torch.cuda.set_per_process_memory_fraction(...)调用,allocator_guard_mentions为 0。common.py、各 benchmark 入口和check_environment.py都未实际施加 23,552 MiB limit,但memory_evidence.json和 README 却声明已设置。当前节点总显存为 49,140 MiB,因此仅凭峰值恰好低于 23 GiB 不能证明在 24GB 预算内可复现。请在每个独立进程首次 CUDA allocation 前实际设置 limit,记录 applied-before-first-allocation、fraction、开跑前空闲显存及失败状态,并重跑正式矩阵、更新 CSV/metadata/memory evidence/图与 README。 - 四个 CSV 使用 CRLF,
git diff --check报告 181 行 trailing whitespace:attention 19、checkpoint 8、compile 9、Flash 145。请统一转换为 LF,保持数据值不变。
使用 48GB 节点本身可按此前放宽口径透明披露,但前提是 23 GiB guard 确实在首次分配前生效;当前证据不满足这一前提。
审核者:GPT-5.6 Sol Ultra
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.
基本信息
修改范围
[A编号] 姓名 - 简短说明。公开性与安全
双层提交
A1/A0 额外检查
不适用:本 PR 是 A2-K。
A2-K 额外检查
实验摘要
给助教的说明
根目录 README 已将 A2-K 标为“已发布”,但 A2-K 题面和 CONTRIBUTING.md 仍保留“发布候选稿/请勿提交”文字;请以课程当前通知为准。
Feishu supplement: https://fudan-nlp.feishu.cn/wiki/QvEmwHImPibuSMkYxbucPKzOn0e