Skip to content

Latest commit

 

History

History
136 lines (95 loc) · 5.01 KB

CONTRIBUTING.md

File metadata and controls

136 lines (95 loc) · 5.01 KB

Contributing code

How to contribute

The preferred way to contribute to nussl is to fork the main repository on GitHub:

  1. Fork the project repository: click on the 'Fork' button near the top of the page. This creates a copy of the code under your account on the GitHub server.

  2. Clone this copy to your local disk:

       $ git clone [email protected]:YourLogin/nussl.git
       $ cd nussl 
    
  3. Create a branch to hold your changes:

       $ git checkout -b my-feature
    

    and start making changes. Never work in the master branch!

  4. Work on this copy on your computer using Git to do the version control. When you're done editing, do:

       $ git add modified_files
       $ git commit
    

    to record your changes in Git, then push them to GitHub with:

       $ git push -u origin my-feature
    

Finally, go to the web page of the your fork of the nussl repo, and click 'Pull request' to send your changes to the maintainers for review. This will send an email to the committers. The committers will probably

(If any of the above seems like magic to you, then look up the Git documentation on the web.)

It is recommended to check that your contribution complies with the following rules before submitting a pull request:

  • All public methods should have informative docstrings with sample usage presented.

  • The documentation follows the Google Python Style guide.

  • The coding style complies with PEP8. Here is a good guide on coding sytle in python.

  • If you are contributing an algorithm that is not your own, we ask that you:

    1. Get permission from the algorithm's original author in writing that it is okay to include in nussl, and
    2. Include original or reference code (and an example audio file) in the tests directory so that we may benchmark it.
  • Write tests for your new code!

You can also check for common programming errors with the following tools:

  • No pyflakes warnings, check with:

        $ pip install pyflakes
        $ pyflakes path/to/module.py
    
  • No PEP8 warnings, check with:

        $ pip install pep8
        $ pep8 path/to/module.py
    
  • AutoPEP8 can help you fix some of the easy redundant errors:

        $ pip install autopep8
        $ autopep8 path/to/pep8.py
    

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.

  • Include all relevant code and/or audio files so that the developers can reproduce your issue. Reproducible bugs are nussl developers' second favorite things (the first is ice cream).

  • Please ensure all code snippets and error messages are formatted in appropriate code blocks. See Creating and highlighting code blocks.

  • Please include your operating system type and version number, as well as your Python, scikit-learn, numpy, scipy, librosa, and nussl versions. This information can be found by runnning the following code snippet:

import platform; print(platform.platform())
import sys; print("Python", sys.version)
import numpy; print("NumPy", numpy.__version__)
import scipy; print("SciPy", scipy.__version__)
import librosa; print("librosa", librosa.__version__)
import nussl; print("nussl", nussl.__version__)

Documentation

Documentation is only in the gh-pages branch. After switching to the gh-pages branch, you can edit the documentation using any text editor and then generate the HTML output by typing make html from the docs/ directory. The resulting HTML files will be placed in _build/html/ and are viewable in a web browser. See this wiki page for more information. Documentation in nussl follows the Google Python Style guide.

For building the documentation, you will need sphinx, and matplotlib.

Note

This document was gleefully borrowed from librosa (which was gleefully borrowed from scikit-learn).