CONTRIBUTING.md 2.22 KB
Newer Older
Zaida Zhou's avatar
Zaida Zhou committed
1
## Contributing to OpenMMLab
Kai Chen's avatar
Kai Chen committed
2

limm's avatar
limm committed
3
All kinds of contributions are welcome, including but not limited to the following.
Kai Chen's avatar
Kai Chen committed
4

limm's avatar
limm committed
5
6
7
- Fix typo or bugs
- Add documentation or translate the documentation into other languages
- Add new features and components
Kai Chen's avatar
Kai Chen committed
8

limm's avatar
limm committed
9
### Workflow
Kai Chen's avatar
Kai Chen committed
10

limm's avatar
limm committed
11
12
13
14
1. fork and pull the latest OpenMMLab repository
2. checkout a new branch (do not use master branch for PRs)
3. commit your changes
4. create a PR
Kai Chen's avatar
Kai Chen committed
15

limm's avatar
limm committed
16
17
```{note}
If you plan to add some new features that involve large changes, it is encouraged to open an issue for discussion first.
18
```
limm's avatar
limm committed
19

limm's avatar
limm committed
20
### Code style
21

limm's avatar
limm committed
22
#### Python
23

limm's avatar
limm committed
24
We adopt [PEP8](https://www.python.org/dev/peps/pep-0008/) as the preferred code style.
25

limm's avatar
limm committed
26
We use the following tools for linting and formatting:
27

limm's avatar
limm committed
28
- [flake8](https://github.com/PyCQA/flake8): A wrapper around some linter tools.
limm's avatar
limm committed
29
- [isort](https://github.com/timothycrosley/isort): A Python utility to sort imports.
limm's avatar
limm committed
30
31
32
- [yapf](https://github.com/google/yapf): A formatter for Python files.
- [codespell](https://github.com/codespell-project/codespell): A Python utility to fix common misspellings in text files.
- [mdformat](https://github.com/executablebooks/mdformat): Mdformat is an opinionated Markdown formatter that can be used to enforce a consistent style in Markdown files.
limm's avatar
limm committed
33
- [docformatter](https://github.com/myint/docformatter): A formatter to format docstring.
34

limm's avatar
limm committed
35
Style configurations of yapf and isort can be found in [setup.cfg](./setup.cfg).
36

limm's avatar
limm committed
37
38
39
We use [pre-commit hook](https://pre-commit.com/) that checks and formats for `flake8`, `yapf`, `isort`, `trailing whitespaces`, `markdown files`,
fixes `end-of-files`, `double-quoted-strings`, `python-encoding-pragma`, `mixed-line-ending`, sorts `requirments.txt` automatically on every commit.
The config for a pre-commit hook is stored in [.pre-commit-config](./.pre-commit-config.yaml).
40

limm's avatar
limm committed
41
After you clone the repository, you will need to install initialize pre-commit hook.
42
43
44
45
46

```shell
pip install -U pre-commit
```

limm's avatar
limm committed
47
From the repository folder
48
49

```shell
limm's avatar
limm committed
50
pre-commit install
51
52
```

limm's avatar
limm committed
53
After this on every commit check code linters and formatter will be enforced.
Kai Chen's avatar
Kai Chen committed
54

limm's avatar
limm committed
55
> Before you create a PR, make sure that your code lints and is formatted by yapf.
56

57
#### C++ and CUDA
58

59
We follow the [Google C++ Style Guide](https://google.github.io/styleguide/cppguide.html).