9. Contributing to pmdarima¶
Note: This document is a ‘getting started’ summary for contributing code, documentation, testing, and filing issues. Please read it carefully to help make the code review process go as smoothly as possible and maximize the likelihood of your contribution being merged.
9.1. How to contribute¶
The preferred workflow for contributing to pmdarima is to fork the main repository on Github, clone, and develop on a branch. Steps:
Fork the project repository by clicking on the ‘Fork’ button near the top right of the page. This creates a copy of the code under your Github user account.
Clone your fork of the pmdarima repo from your Github account to your local disk:
$ git clone https://github.com/alkaline-ml/pmdarima.git $ cd pmdarima
Create a
feature
branch to hold your development changes:$ git checkout -b my-feature
Always use a
feature
branch. It’s good practice to never work on themaster
branch!Develop the feature on your feature branch. Add changed files using
git add
and thengit commit
files:$ git add modified_files $ git commit
to record your changes in Git, then push the changes to your Github account with:
$ git push -u origin my-feature
5. Follow these instructions to create a pull request from your fork. This will send an email to the committers.
9.2. Pull Request Checklist¶
We recommended (and prefer that) that your contribution complies with the following rules before you submit a pull request. Failure to adhere to the rules may hinder the speed with which your contribution is merged:
Write detailed docstrings for all of your public functions. The preferred format for docstrings is the numpy standard. Also include usage examples where appropriate. See also the Numpy guidelines for documenting your code
Use, when applicable, the validation tools and scripts in the
pmdarima.utils
submodule.Give your merge request a helpful title that summarizes what your contribution does. In some cases
Fix <ISSUE TITLE>
is enough.Fix #<ISSUE NUMBER>
is not enough.If your pull request references an issue, reference it in the body of your descriptive text using
#<ISSUE NUMBER>
Please prefix the title of your pull request with
[MRG]
(Ready for Merge), if the contribution is complete and ready for a detailed review. The core developers will then review your code and merge when approved. An incomplete contribution – where you expect to do more work before receiving a full review – should be prefixed[WIP]
(to indicate a work in progress) and changed to[MRG]
when it matures. WIPs may be useful to: indicate you are working on something to avoid duplicated work, request broad review of functionality or API, or seek collaborators.All other tests pass when everything is rebuilt from scratch. On Unix-like systems, check with (from the toplevel source folder):
$ python setup.py develop $ pytest
You can also use the
Makefile
on posix machines:$ make test
You may need to see the Setup section for instructions on how to build the package. For instructions on how to test (using pytest) see Numpy’s testing instructions.
9.3. Filing bugs¶
We use Github issues to track all bugs and feature requests; feel free to open an issue if you have found a bug or wish to see a feature implemented.
It is recommended to check that your issue complies with the following rules before submitting:
- Verify that your issue is not being currently addressed by other issues or pull requests.
- If your issue references and pull request, reference it in the body of your
descriptive text using
!<PULL REQUEST NUMBER>
- Please include your operating system type and version number, as well as your Python, statsmodels, scikit-learn, numpy, scipy, pandas and pmdarima versions. This information can be found by running the following code snippet:
import pmdarima as pm; pm.show_versions()
- Please don’t be a lazy issue-filer! Submitting a screen shot of an Excel document, or poorly-formatted/incomplete code makes the maintainers’ lives difficult. Please include your data inline in a code-block so maintainers can easily try to replicate. What not to do:
A better way to file the same issue (made up; this issue was not actually filed):