Contributing#

Contributions are welcome and greatly appreciated!

The best way to get in touch with the core developers and maintainers is to join the PyLops slack channel as well as open new Issues directly from the GitHub repo.

Welcomed contributions#

Bug reports#

Report bugs at PyLops/pylops-mpi#issues.

If you are playing with the PyLops-MPI library and find a bug, please report it including:

  • Your operating system name and version.

  • Any details about your Python environment.

  • Detailed steps to reproduce the bug.

New operators and features#

Open an issue at PyLops/pylops-mpi#issues with tag enhancement.

If you are proposing a new operator or a new feature:

  • Explain in detail how it should work.

  • Keep the scope as narrow as possible, to make it easier to implement.

Fix issues#

There is always a backlog of issues that need to be dealt with. Look through the GitHub Issues for operator/feature requests or bugfixes.

Add examples or improve documentation#

Writing new operators is not the only way to get involved and contribute. Create examples with existing operators as well as improving the documentation of existing operators is as important as making new operators and very much encouraged.

Step-by-step instructions for contributing#

Ready to contribute?

  1. Follow all instructions in Step-by-step installation for developers.

  2. Create a branch for local development, usually starting from the main branch:

>> git checkout -b name-of-your-branch

Now you can make your changes locally.

3. When you’re done making changes, check that your code follows the guidelines for Implementing new operators and that the both old and new tests pass successfully:

>> make tests

  1. Make sure the examples python scripts are executed using 3 processes without any errors:

>> make run_examples

  1. Make sure the tutorials python scripts are executed using 3 processes without any errors:

>> make run_tutorials

  1. Run flake8 to check the quality of your code:

>> make lint

Note that PyLops-MPI does not enforce full compliance with flake8, rather this is used as a guideline and will also be run as part of our CI. Make sure to limit to a minimum flake8 warnings before making a PR.

  1. Build the docs:

>> make doc

  1. Commit your changes and push your branch to GitHub:

>> git add .
>> git commit -m "Your detailed description of your changes."
>> git push origin name-of-your-branch

Remember to add -u when pushing the branch for the first time.

  1. Submit a pull request through the GitHub website.

Pull Request Guidelines#

Before you submit a pull request, check that it meets these guidelines:

  1. The pull request should include new tests for all the core routines that have been developed.

  2. If the pull request adds functionality, the docs should be updated accordingly.

  3. Ensure that the updated code passes all tests.

Project structure#

This repository is organized as follows:

  • pylops_mpi: Python library containing various mpi linear operators.

  • tests: Set of tests using pytest-mpi.

  • testdata: Sample datasets used in tests and documentation.

  • docs: Sphinx documentation.

  • examples: Set of python script examples for each mpi linear operator to be embedded in documentation using sphinx-gallery.

  • tutorials: Set of python script tutorials to be embedded in documentation using sphinx-gallery.