Contributing#
Contributions are welcome and greatly appreciated!
The best way to get in touch with the core developers and mantainers is to open new Issues directly from the github repo.
Welcomed contributions#
Bug reports#
Report bugs at PyLops/pyproximal#issues.
If you are playing with the PyProximal 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/pyproximal#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?
Follow all instructions in Step-by-step installation for developers.
Create a branch for local development, usually starting from the dev 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 both old and new tests pass successfully:
>> make tests
Run flake8 to check the quality of your code:
>> make lint
Note that PyProximal 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.
Update the docs
>> make docupdate
Commit your changes and push your branch to GitLab:
>> 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.
We recommend using Conventional Commits to
format your commit messages, but this is not enforced.
Submit a pull request through the GitHub website.
Pull Request Guidelines#
Before you submit a pull request, check that it meets these guidelines:
The pull request should include new tests for all the core routines that have been developed.
If the pull request adds functionality, the docs should be updated accordingly.
Ensure that the updated code passes all tests.
Project structure#
This repository is organized as follows:
pyproximal: Python library containing various proximal operators and solvers
pytests: set of pytests
testdata: sample datasets used in pytests and documentation
docs: Sphinx documentation
examples: set of python script examples for each proximal operator to be embedded in documentation using sphinx-gallery
tutorials: set of python script tutorials to be embedded in documentation using sphinx-gallery