Skip to content

添加2025_starAndHonor笔记:复变函数、大学物理(二)、机器学习、计算机组成原理 - #33

Merged
Snowflake-Pink merged 1 commit into
OpenFuTech:mainfrom
starAndHonor:add-notes-2025-starAndHonor
Sep 7, 2026
Merged

Snowflake-Pink merged 1 commit into
OpenFuTech:mainfrom
starAndHonor:add-notes-2025-starAndHonor

Conversation

@starAndHonor

Copy link
Copy Markdown
Contributor

按贡献规范提交四份个人课程笔记(均为原创笔记 PDF,单个文件均小于 100M):

  • 01-基础必修课/复变函数/note/2025_starAndHonor/部分复变函数笔记.pdf
  • 01-基础必修课/大学物理(二)/note/2025_starAndHonor/大学物理笔记.pdf
  • 02-专业基础课/机器学习/notes/2025_starAndHonor/机器学习笔记.pdf(沿用该课程现有 notes/ 目录)
  • 02-专业基础课/计算机组成与体系结构/note/2025_starAndHonor/计算机组成原理笔记.pdf(新建 note/ 目录)

@starAndHonor
starAndHonor requested a review from a team September 6, 2026 10:19

@xinxiangchen xinxiangchen 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.

格式正确,不保证pr同学笔记内容完全无误

@Snowflake-Pink

Copy link
Copy Markdown
Contributor

/oc

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

PR #33 审查结果

已通读 Contribution.md 并对照仓库现有课程目录结构核查了本次提交的 4 个文件(均为有效 PDF 文件,文件头已验证)。

重点建议修改

无。四个文件均符合规范:

文件 路径规范 大小
部分复变函数笔记.pdf 复变函数/note/2025_starAndHonor/ ✅ 0.3 MB
大学物理笔记.pdf 大学物理(二)/note/2025_starAndHonor/ ✅ 1.6 MB
机器学习笔记.pdf 机器学习/notes/2025_starAndHonor/ ✅(见下) 1.4 MB
计算机组成原理笔记.pdf 计算机组成与体系结构/note/2025_starAndHonor/ ✅(新建目录,正确) 2.3 MB

均按 2025_学生昵称/ 组织,无重复下载痕迹、无垃圾文件混入,原创笔记也无版权问题。文件大小远低于 50 MB,对仓库体积无压力。

其他建议修改

无。

可选

  1. 机器学习 课程沿用了 notes/ 拼写(02-专业基础课/机器学习/notes/2025_starAndHonor/机器学习笔记.pdf):规范原文是 note/,但该课程在 main 分支上已有的 notes/机器学习笔记.pdf 就是复数形式,且资源列表由脚本(update.py)全量自动收录,不区分单复数,所以功能上没有任何影响。沿用现状完全可以接受;若你熟悉 git 操作、愿意顺手把新旧文件统一迁移到 note/ 就更好了(涉及重命名,网页端做不了,不勉强)。
  2. 欢迎补充文字攻略:本次只提交了笔记 PDF,如果之后愿意,也可以在对应课程 README.md 的「修读攻略」等标题下补几段文字(记得用 2025级 starAndHonor: 的署名另起段落),对后来者会更有帮助,但不是必须。

结论

✅ 通过。本 PR 干净规范,无阻塞问题,感谢贡献!以上「可选」条目留给维护者和你自行斟酌即可。

github run

@Snowflake-Pink
Snowflake-Pink merged commit 02e047c into OpenFuTech:main Sep 7, 2026
1 check passed
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.

3 participants