-
Notifications
You must be signed in to change notification settings - Fork 2k
Contributing and Code Style
Jackwaterveg edited this page Jan 25, 2022
·
33 revisions
- 可读性
第一原则 - 风格统一
C++ 与 Kaldi 风格一致
Code Style- python: precommit run -a
- c++: clang-format, google style
docstring: 使用 Python Docstring Generator 插件,默认用 Google (default)
风格转换工具:https://github.com/dadadel/pyment/
- 可维护性
- 先将 plan 在 project中发布,plan 统一风格
- 开工前发布 issue
- 遗留项加入 project 的 plan
- 工作开始于此,结束于此
- 小步快跑
- 大PR只有大 Feature 才可以
- 开工前先提 PR 便于协作和 review
- 每次PR需要填上:Project/Milestone 信息
- 如有关联的issue:
Fix #333 - 提交 PR 前请 pre-commit or clang-format
- 在 repo 的 Projects 中选择需要修改的 project,如PaddleSpeech。
- 进入 paddlespeech,可以看到 The plan(Todo list)。在左下角处添加Todo内容(写好题目后回车即可)。添加好以后,注意补全 Assignees, Status 和 milestone 信息。milestone 添加的时候可以转变为 issue,在 repo 的 issues 中会自动建立这个 Todo。
- 提交 PR,注意在提交的信息中(不是标题)添加"Fix #xxx" (xxx表示 issue 代码)。
- Close issue,并且在 Todo list 中把对应 Todo 的状态改为 Done。
- PaddleSpeech/PaddleAudio 下 CHANGELOG.md
- 按 Feature 添加,样例如下。需要清晰支持什么时间,加了什么功能,以便后续复盘使用。
日期-【新增】Conformer ASR model