Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

提交第四章实验报告 #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

提交第四章实验报告 #4

wants to merge 1 commit into from

Conversation

Lychee00
Copy link
Collaborator

No description provided.

@chencwx chencwx self-requested a review November 22, 2021 13:47
chencwx
chencwx approved these changes Nov 22, 2021
View reviewed changes
Copy link

@chencwx chencwx left a comment

Choose a reason for hiding this comment

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

Accepted

Pros

  • 完成可选实验，好评👍
  • 基本完成实验要求
  • 在Pr标题中体现了作业序号
  • 推荐的目录结构
  • 见名知意的文件命名
  • 实验报告结构规范，内容较完整
  • 作业中有适当的「身份标识」证明作业原创性
  • 有基本的配图与高亮标记
  • 直接在配图中对高亮标记进行文字标注
  • 对实验结果给出了自己的解释
  • 在实验报告末总结了遇到的问题与解决方案
  • 参考文献/资料引用标准规范

Cons

  • 「问题与解决方法」中的图片渲染换行失败了，建议下次pr之前自查一遍

Recommends

  • 实验报告过程很完整清晰，如果能继续完成课本中的「实验三、实验四」那就更好啦

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

@chencwx chencwx chencwx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Lychee00 @chencwx