Thank you for contributing to osier
! All contributions, bug reports, bug fixes, documentation
improvements, enhancements and ideas are welcome.
We use GitHub "Issues" tab to track bugs and feature requests. Issues labeled "Difficulty:1-Beginner" are good first issues for new contributors to tackle. You can sort by "label" to find these issues.
Bug reports from users are important for identifying unintentional behavior. If you find a bug, please open a new issue with the following:
- An explanation of the problem with enough details for others to reproduce the problem. Some
common information needed is:
- Operating system
- Python version
- Any commands executed (perhaps a python snippet)
- An error message from the terminal
- An explanation of the expected behavior. For example:
- I ran
numpy.add(1,2)
which gave me an output of-999
, but I expected3
.
- I ran
In order to add new features to osier
you need to set up a working development environment.
First, you must create a fork. Then use the following
commands. "your-fork" is a placeholder for the location of your fork of osier
.
# 1. Download source code
git clone [email protected]:your-fork/osier.git # requires ssh-keys
# or
git clone https://github.com/your-fork/osier.git
# 2. Open the osier directory
cd osier
# 3. Add a remote to the official repository
git remote add arfc https://github.com/arfc/osier.git # the official repository
# 4. Update osier
git pull arfc main # pull down the up-to-date version of osier
# 5. Install editable configuration
pip install -e .[doc]
The -e
flag creates an editable installation so you don't have to reinstall everytime a
feature is changed.
If you encounter an error
zsh: no matches found: .[doc]
running the final pip installation command for the docs, other users have remedied the error by running:
pip install -e .'[doc]'
A good pull request requires the following, along with a new feature (where applicable)
- All functions should have docstrings using the Numpydoc style.
- All new functions should have corresponding unit tests (and should be small enough that unit-testing makes sense).
- All tests must pass on your machine by running
pytest
in the top level directory. - All new features must be appropriately documented.
- Code should follow PEP8 style.
autopep8
is a helpful tool for ensuring consistent style.