CONTRIBUTING.md 1.58 KB
Newer Older
1
# Contributing to MMCV
Kai Chen's avatar
Kai Chen committed
2
3
4
5
6
7
8
9

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

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

## Workflow

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

Note: If you plan to add some new features that involve large changes, it is encouraged to open an issue for discussion first.

## Code style

### Python
Kai Chen's avatar
Kai Chen committed
20

Kai Chen's avatar
Kai Chen committed
21
22
23
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
24

Kai Chen's avatar
Kai Chen committed
25
26
27
28
- [flake8](http://flake8.pycqa.org/en/latest/): linter
- [yapf](https://github.com/google/yapf): formatter
- [isort](https://github.com/timothycrosley/isort): sort imports

29
30
31
32
33
34
35
36
Style configurations of yapf and isort can be found in [setup.cfg](./setup.cfg).

We use [pre-commit hook](https://pre-commit.com/) that checks and formats for `flake8`, `yapf`, `isort`, `trailing whitespaces`,
 fixes `end-of-files`, sorts `requirments.txt` automatically on every commit.
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
37
```shell
38
39
40
41
pip install -U pre-commit
```

From the repository folder
Kai Chen's avatar
Kai Chen committed
42
43

```shell
44
45
46
47
48
pre-commit install
```

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

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

### C++ and CUDA
Kai Chen's avatar
Kai Chen committed
52

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