Skip to content

Latest commit

 

History

History
119 lines (79 loc) · 3.65 KB

CONTRIBUTING.md

File metadata and controls

119 lines (79 loc) · 3.65 KB
Raw

Contributing to Stable-Baselines3

If you are interested in contributing to Stable-Baselines, your contributions will fall into two categories:

  1. You want to propose a new Feature and implement it
    • Create an issue about your intended feature, and we shall discuss the design and implementation. Once we agree that the plan looks good, go ahead and implement it.
  2. You want to implement a feature or bug-fix for an outstanding issue

Once you finish implementing a feature or bug-fix, please send a Pull Request to https://github.com/DLR-RM/stable-baselines3

If you are not familiar with creating a Pull Request, here are some guides:

Developing Stable-Baselines3

To develop Stable-Baselines3 on your machine, here are some tips:

  1. Clone a copy of Stable-Baselines3 from source:
git clone https://github.com/DLR-RM/stable-baselines3
cd stable-baselines3/
  1. Install Stable-Baselines3 in develop mode, with support for building the docs and running tests:
pip install -e .[docs,tests,extra]

Codestyle

We are using black codestyle (max line length of 127 characters) together with isort to sort the imports.

Please run make format to reformat your code. You can check the codestyle using make check-codestyle and make lint.

Please document each function/method and type them using the following template:

def my_function(arg1: type1, arg2: type2) -> returntype:
    """
    Short description of the function.

    :param arg1: describe what is arg1
    :param arg2: describe what is arg2
    :return: describe what is returned
    """
    ...
    return my_variable

Pull Request (PR)

Before proposing a PR, please open an issue, where the feature will be discussed. This prevent from duplicated PR to be proposed and also ease the code review process.

Each PR need to be reviewed and accepted by at least one of the maintainers (@hill-a, @araffin, @ernestum, @AdamGleave or @Miffyli). A PR must pass the Continuous Integration tests to be merged with the master branch.

Tests

All new features must add tests in the tests/ folder ensuring that everything works fine. We use pytest. Also, when a bug fix is proposed, tests should be added to avoid regression.

To run tests with pytest:

make pytest

Type checking with pytype:

make type

Codestyle check with black, isort and flake8:

make check-codestyle
make lint

To run pytype, format and lint in one command:

make commit-checks

Build the documentation:

make doc

Check documentation spelling (you need to install sphinxcontrib.spelling package for that):

make spelling

Changelog and Documentation

Please do not forget to update the changelog (docs/misc/changelog.rst) and add documentation if needed. You should add your username next to each changelog entry that you added. If this is your first contribution, please add your username at the bottom too. A README is present in the docs/ folder for instructions on how to build the documentation.

Credits: this contributing guide is based on the PyTorch one.