CONTRIBUTING.md 2.44 KB
Newer Older
Zaida Zhou's avatar
Zaida Zhou committed
1
## Contributing to OpenMMLab
Kai Chen's avatar
Kai Chen committed
2
3
4
5
6
7

All kinds of contributions are welcome, including but not limited to the following.

- Fixes (typo, bugs)
- New features and components

Zaida Zhou's avatar
Zaida Zhou committed
8
### Workflow
Kai Chen's avatar
Kai Chen committed
9

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

Zaida Zhou's avatar
Zaida Zhou committed
15
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.
```
Zaida Zhou's avatar
Zaida Zhou committed
18
### Code style
Kai Chen's avatar
Kai Chen committed
19

Zaida Zhou's avatar
Zaida Zhou committed
20
#### Python
Kai Chen's avatar
Kai Chen committed
21

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

We use the following tools for linting and formatting:
Kai Chen's avatar
Kai Chen committed
25

26
27
28
29
30
- [flake8](http://flake8.pycqa.org/en/latest/): A wrapper around some linter tools.
- [yapf](https://github.com/google/yapf): A formatter for Python files.
- [isort](https://github.com/timothycrosley/isort): A Python utility to sort imports.
- [markdownlint](https://github.com/markdownlint/markdownlint): A linter to check markdown files and flag style issues.
- [docformatter](https://github.com/myint/docformatter): A formatter to format docstring.
Kai Chen's avatar
Kai Chen committed
31

32
33
Style configurations of yapf and isort can be found in [setup.cfg](./setup.cfg).

34
35
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.
36
37
38
39
The config for a pre-commit hook is stored in [.pre-commit-config](./.pre-commit-config.yaml).

After you clone the repository, you will need to install initialize pre-commit hook.

Kai Chen's avatar
Kai Chen committed
40
```shell
41
42
43
44
pip install -U pre-commit
```

From the repository folder
Kai Chen's avatar
Kai Chen committed
45
46

```shell
47
48
49
pre-commit install
```

50
51
52
53
54
55
56
57
58
59
60
61
62
63
Try the following steps to install ruby when you encounter an issue on installing markdownlint

```shell
# install rvm
curl -L https://get.rvm.io | bash -s -- --autolibs=read-fail
[[ -s "$HOME/.rvm/scripts/rvm" ]] && source "$HOME/.rvm/scripts/rvm"
rvm autolibs disable

# install ruby
rvm install 2.7.1
```

Or refer to [this repo](https://github.com/innerlee/setup) and take [`zzruby.sh`](https://github.com/innerlee/setup/blob/master/zzruby.sh) according its instruction.

64
65
After this on every commit check code linters and formatter will be enforced.

Kai Chen's avatar
Kai Chen committed
66
67
>Before you create a PR, make sure that your code lints and is formatted by yapf.

Zaida Zhou's avatar
Zaida Zhou committed
68
#### C++ and CUDA
Kai Chen's avatar
Kai Chen committed
69

Kai Chen's avatar
Kai Chen committed
70
We follow the [Google C++ Style Guide](https://google.github.io/styleguide/cppguide.html).