Skip to content

[A2-k] 张俊鹏 - 第二次提交 - #153

Open
lalalalulu2 wants to merge 3 commits into
OpenMOSS:mainfrom
lalalalulu2:a2-k/lalalalulu2
Open

[A2-k] 张俊鹏 - 第二次提交#153
lalalalulu2 wants to merge 3 commits into
OpenMOSS:mainfrom
lalalalulu2:a2-k/lalalalulu2

Conversation

@lalalalulu2

Copy link
Copy Markdown
Contributor

PR Checklist

基本信息

  • 同学真名:
  • 作业编号:A0

修改范围

  • 本 PR 只包含我本人本次作业的文件。
  • 我没有修改其他同学、students/_template、公共题面或仓库配置。
  • PR 标题符合 [A编号] 姓名 - 简短说明

公开性与安全

  • 我理解 GitHub 中所有内容均为公开资料。
  • 公开内容不含内部主机名、IP、账号、路径、数据或未公开项目。
  • GitHub 和飞书正文均不包含 Secret、Token、Cookie、密码或私钥。
  • 我检查了 git diff --cached,没有提交大型数据、模型权重、缓存或完整日志。

双层提交

  • README.md 提供可公开、已脱敏的作业报告。
  • README.md 已填写飞书补充文档链接。

A1 额外检查

若本 PR 不是 A1,请将本节标记为不适用。

  • 已提交 README.mdsubmission/cs336_basics/submission/tests/adapters.py
    submission/scripts/logs/
  • 报告文件为 README.md(Markdown)。
  • assignment1-basics 工作仓库位于 SummerQuest 的兄弟目录,且没有被提交到本 PR。
  • 已在 ../assignment1-basics 运行官方测试,并使用同步脚本更新个人提交目录。

A0 额外检查

若本 PR 不是 A0,请将本节标记为不适用。

  • 已完成公开 GitHub profile 和组内飞书 profile。
  • 已在个人服务器实际运行 nvidia-smigpustat 并记录退出码。
  • A0 的组内验收材料已放入 README 链接的飞书补充文档。

A2-P 额外检查

若本 PR 不是 A2-P,请将本节标记为不适用。A2-P 已正式发布,按题面要求提交。

  • 已提交 Markdown README.mdsubmission/profiling/**/*.py、规定的轻量
    results/ 和至少三张被报告引用的裁剪、压缩图片。
  • 已运行 scripts/sync_a2p_submission.pyscripts/validate_repo.py
  • 已使用 nsys 或 torch.profiler 完成两个模型规模 × 三个 context 的六个
    train_step trace。
  • 仓库外源码与资料引用使用固定 commit 的 GitHub HTTPS 绝对 URL,未使用本机或跨仓库相对链接。
  • 未提交 .nsys-rep、snapshot、完整 trace、权重、数据、压缩包、上游代码或依赖环境。
  • results/assets/ 合计不超过 2 MiB,Profile 只保留关键汇总与截图。

A2-K 额外检查

若本 PR 不是 A2-K,请将本节标记为不适用。A2-K 已正式发布,按题面要求提交。

  • 已提交 Markdown README.md、A2-K Python allowlist、全部规定的轻量 results/
    和至少两张被报告引用的压缩图片。
  • 已运行 scripts/sync_a2k_submission.py、官方 GPU tests 和
    scripts/validate_repo.py,且没有把 CUDA skip 写成 pass。
  • 正式矩阵来自单张 RTX 4090 24GB;各脚本独立串行执行,并在首次 CUDA allocation
    前设置 23552 MiB allocator 上限。
  • memory_evidence.json 已记录 allocator limit/fraction、peak allocated/reserved 和
    24 GiB 判定。
  • 仓库外源码与资料引用使用固定 commit 的 GitHub HTTPS 绝对 URL,未使用本机或跨仓库相对链接。
  • 未提交 compile cache、PTX/CUBIN、binary、完整日志/采样、权重、数据、压缩包、上游代码或依赖环境。
  • results/assets/ 合计不超过 2 MiB,关键数字可回到轻量结果与命令。

给助教的说明

<说明未完成项、环境限制或希望重点审核的内容。不要粘贴密钥和内部地址。>

@WillQvQ WillQvQ left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审核结论:需要修改后再合并。

当前有三项明确阻塞:

  1. PR 同时新增 students/张俊鹏/assignments/A2-K/ 与完整的 students/张俊鹏/assignments/A2-P/。仓库要求一个 PR 只修改一名同学的一项作业;请将 A2-P 拆到独立 PR,本 PR 只保留 A2-K。
  2. 标题应严格使用 [A2-K] 张俊鹏 - 简短说明,当前 [A2-k] 大小写不符合约定;PR 描述中的“作业编号:A0”、空白基本信息和未填写 checklist 也应更新为本次 A2-K 的真实状态。
  3. 当前附带的 A2-P CSV 使用 CRLF,导致 git diff --check 报告大量 trailing whitespace;拆分 A2-P 时也请统一为 LF。

A2-K 本身的 Triton 实现、6 项 GPU test、正确性记录、矩阵与 23 GiB allocator guard 已完成静态核验;完成上述范围与元数据修复后可继续复核。

审核者:GPT-5.6 Sol Ultra

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants