This repository has been archived by the owner on Jan 24, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 24
📚 PaddleSOT 源码阅读 CRC 快乐喵喵之路 🎉 #186
Labels
HappyOpenSource
快乐开源活动issue与PR
Comments
认领 1、14、15 |
28 29 30 |
认领21、16 |
认领 23 24 25 |
认领6.11 |
认领26 |
认领7-10 |
This was referenced Jun 23, 2023
This was referenced Jun 24, 2023
This was referenced Jul 1, 2023
认领11 |
认领36 |
This was referenced Aug 7, 2023
github-project-automation
bot
moved this from In Progress
to Done
in Call for Contributions
Aug 8, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
一、Background 📚
自6月初 PaddleSOT 第一次与各位小伙伴完成技术分享和讨论之后,发布了PaddleSOT 快乐喵喵开源任务,真切感受到了小伙伴们的热情,也陆续收到了 @zrr1999 、@gouzil、@Difers、@GreatV、@NotHaozi 等同学贡献的 PR 👏🏻。
二、Motivation 🚀
为了进一步营造 PaddleSOT 快乐开源的的社区氛围,与各位小伙伴们一同学习、成长,我们计划开启「新·篇章」,即 PaddleSOT 源码阅读 Code Reading and Comment (简称 CRC)快乐喵喵之路。
在这里,小伙伴们可以一起阅读 PaddleSOT 的源码,并鼓励提出自己的疑惑到 Disscussions ,我们会不定期组织答疑分享。我们将以 「✏️ CRC 代码标注赛 ✏️」 的形式来组织:
Class
或者Funciton
下贡献自己的「理解注释」,欢迎提交 PR 给 PaddleSOT 🤩通过 「CRC 快乐喵喵之路」,小伙伴可以:
三、 CRC Tasks 🙋🏻♀️ (整体进展:37/37)
tracker.py
)base.py
)basic.py
)basic.py
))callable.py
)callable.py
)callable.py
)container.py
)tracker.py
)tracker.py
)四、Comment 规范 🌲
4.1 Class 的注释
对于核心模块的
Class
,一般可以考虑从如下几个维度来添加注解:如下是框架的一个参考样例:
4.2 函数的注释
对于核心模块的
Function
或者类的method
,一般从如下几个维度来添加注释:如下是框架的一个参考样例:
4.3 类型提示
我们代码里已经包含了大量的类型提示,但是部分早期的代码类型提示信息仍然不完整,因此希望在完善注释的同时完善类型提示信息:
应当注意仅类型提示的第一方模块在引用时应尽可能放在
if TYPE_CHECKING:
下,以消除运行时影响,降低循环依赖的可能性,比如:PaddleSOT/sot/opcode_translator/executor/opcode_inline_executor.py
Lines 13 to 15 in d0ede5e
如非必要应当尽可能使用
list
等容器类型而非List
,使用不带引号的类别名而非带引号的类别名:详情可参考 typing docs、PEP 484
The text was updated successfully, but these errors were encountered: