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.

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

Chap0x04实验报告提交 #4

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Chap0x04实验报告提交 #4

wants to merge 5 commits into from

Conversation

guoxingzi
Copy link
Collaborator

Chap0x04实验报告提交

@pigeoner
Copy link

pigeoner commented Jan 6, 2023

Accepted

Pros

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

Cons

  • 根据你的实验报告中列出的代码,如果没有在 ARP 数据包在外面加一层 Ethernet 帧头 Ether() 或使用 send 代替 sendp,ARP 数据包是无法正常发送的,参考这位同学的实验报告
  • 所有图片全部渲染失败,原因是文件路径符使用了\而不是/
  • 第二个实验的内容没有任何实验过程截图
  • 正如你在反思总结中所言,本次实验只是机械地按步骤走了一遍流程,没有体现出自己的思考
  • 没有身份标识符
  • 作业提交时间太晚

Recommends

  • 认真对待每一次作业,希望你能认真反思,最好重新做一次实验

@guoxingzi
Copy link
Collaborator Author

好的,谢谢老师,我进行了二次实验的小补充

@pigeoner
Copy link

pigeoner commented Jan 6, 2023

(以下为第二次提交内容的批改)

  • 首先,依然没有身份标识。一个比较合理的身份标识方法是在终端键入自己的GitHub账号id或其他能证明身份的内容
  • 其次,好好检查安装scapy部分,你安装的是【scrapy】而不是【scapy】,scrapy是一个爬虫框架,和本次实验没有任何关系。既然你安装错了包,那么后续的实验是如何进行的呢?请给出合理的解释
  • 按照常理来讲,在过了提交期限之后,已评分的作业成绩不能再修改。但如果你能把上述问题解决,我可以对你的成绩稍作调整。

@pigeoner
Copy link

pigeoner commented Jan 7, 2023

补充一点,你的第二次提交的内容和一位同学出现了雷同,如果你还想修改作业内容,在新的提交中必须使用终端键入id的方法体现身份标识

@pigeoner
Copy link

pigeoner commented Jan 7, 2023

Accepted

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