File size: 6,481 Bytes
256a159 |
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 |
# 为 OpenCompass 做贡献
- [为 OpenCompass 做贡献](#为-opencompass-做贡献)
- [什么是拉取请求？](#什么是拉取请求)
- [基本的工作流：](#基本的工作流)
- [具体步骤](#具体步骤)
- [1. 获取最新的代码库](#1-获取最新的代码库)
- [2. 从 `main` 分支创建一个新的开发分支](#2-从-main-分支创建一个新的开发分支)
- [3. 提交你的修改](#3-提交你的修改)
- [4. 推送你的修改到复刻的代码库，并创建一个拉取请求](#4-推送你的修改到复刻的代码库并创建一个拉取请求)
- [5. 讨论并评审你的代码](#5-讨论并评审你的代码)
- [6. `拉取请求`合并之后删除该分支](#6-拉取请求合并之后删除该分支)
- [代码风格](#代码风格)
- [Python](#python)
- [关于提交数据集](#关于提交数据集)
感谢你对于OpenCompass的贡献！我们欢迎各种形式的贡献，包括但不限于以下几点。
- 修改错别字或修复bug
- 添加文档或将文档翻译成其它语言
- 添加新功能和组件
## 什么是拉取请求？
`拉取请求` (Pull Request), [GitHub 官方文档](https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests)定义如下。
```
拉取请求是一种通知机制。你修改了他人的代码，将你的修改通知原来作者，希望他合并你的修改。
```
## 基本的工作流：
1. 获取最新的代码库
2. 从最新的 `main` 分支创建分支进行开发
3. 提交修改 ([不要忘记使用 pre-commit hooks!](#3-提交你的修改))
4. 推送你的修改并创建一个 `拉取请求`
5. 讨论、审核代码
6. 将开发分支合并到 `main` 分支
## 具体步骤
### 1. 获取最新的代码库
- 当你第一次提 PR 时
复刻 OpenCompass 原代码库，点击 GitHub 页面右上角的 **Fork** 按钮即可
克隆复刻的代码库到本地
```bash
git clone [email protected]:XXX/opencompass.git
```
添加原代码库为上游代码库
```bash
git remote add upstream [email protected]:InternLM/opencompass.git
```
- 从第二个 PR 起
检出本地代码库的主分支，然后从最新的原代码库的主分支拉取更新。
```bash
git checkout main
git pull upstream main
```
### 2. 从 `main` 分支创建一个新的开发分支
```bash
git checkout main -b branchname
```
### 3. 提交你的修改
- 如果你是第一次尝试贡献，请在 OpenCompass 的目录下安装并初始化 pre-commit hooks。
```bash
pip install -U pre-commit
pre-commit install
```
````{tip}
对于中国地区的用户，由于网络原因，安装 pre-commit hook 可能会失败。可以尝试以下命令切换为国内镜像源：
```bash
pre-commit install -c .pre-commit-config-zh-cn.yaml
pre-commit run –all-files -c .pre-commit-config-zh-cn.yaml
```
````
- 提交修改。在每次提交前，pre-commit hooks 都会被触发并规范化你的代码格式。
```bash
# coding
git add [files]
git commit -m 'messages'
```
```{note}
有时你的文件可能会在提交时被 pre-commit hooks 自动修改。这时请重新添加并提交修改后的文件。
```
### 4. 推送你的修改到复刻的代码库，并创建一个拉取请求
- 推送当前分支到远端复刻的代码库
```bash
git push origin branchname
```
- 创建一个拉取请求
- 修改拉取请求信息模板，描述修改原因和修改内容。还可以在 PR 描述中，手动关联到相关的议题 (issue),（更多细节，请参考[官方文档](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue)）。
- 你同样可以把 PR 关联给相关人员进行评审。
### 5. 讨论并评审你的代码
- 根据评审人员的意见修改代码，并推送修改
### 6. `拉取请求`合并之后删除该分支
- 在 PR 合并之后，你就可以删除该分支了。
```bash
git branch -d branchname # 删除本地分支
git push origin --delete branchname # 删除远程分支
```
## 代码风格
### Python
我们采用[PEP8](https://www.python.org/dev/peps/pep-0008/)作为首选的代码风格。
我们使用以下工具进行linting和格式化：
- [flake8](https://github.com/PyCQA/flake8): 一个围绕一些linter工具的封装器。
- [isort](https://github.com/timothycrosley/isort): 一个用于排序Python导入的实用程序。
- [yapf](https://github.com/google/yapf): 一个Python文件的格式化器。
- [codespell](https://github.com/codespell-project/codespell): 一个Python实用程序，用于修复文本文件中常见的拼写错误。
- [mdformat](https://github.com/executablebooks/mdformat): mdformat是一个有明确定义的Markdown格式化程序，可以用来在Markdown文件中强制执行一致的样式。
- [docformatter](https://github.com/myint/docformatter): 一个格式化docstring的工具。
yapf和isort的样式配置可以在[setup.cfg](https://github.com/OpenCompass/blob/main/setup.cfg)中找到。
## 关于贡献测试数据集
- 提交测试数据集
- 请在代码中实现自动下载数据集的逻辑；或者在 PR 中提供获取数据集的方法，OpenCompass 的维护者会跟进处理。如果数据集尚未公开，亦请注明。
- 提交数据配置文件
- 在数据配置同级目录下提供 README，README 中的内容应该包含，但不局限于：
- 该数据集的简单说明
- 该数据集的官方链接
- 该数据集的一些测试样例
- 该数据集在相关模型上的评测结果
- 该数据集的引用
- (可选) 数据集的 summarizer
- (可选) 如果测试过程无法通过简单拼接数据集和模型配置文件的方式来实现的话，还需要提供进行测试过程的配置文件
- (可选) 如果需要，请在文档相关位置处添加该数据集的说明。这在辅助用户理解该测试方案是非常必要的，可参考 OpenCompass 中该类型的文档：
- [循环评测](../advanced_guides/circular_eval.md)
- [代码评测](../advanced_guides/code_eval.md)
- [污染评估](../advanced_guides/contamination_eval.md)
|