How to contribute#

Bug report and feature request#

We use benchopt GitHub repo 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.

Code contribution#

The preferred way to contribute to benchopt is to fork the main repository on GitHub, then submit a “Pull Request” (PR).

In the first few steps, we explain how to locally install benchopt, and how to set up your git repository:

  1. Create an account on GitHub if you do not already have one.

  2. 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 user account. For more details on how to fork a repository see this guide.

  3. Clone your fork of the benchopt repo from your GitHub account to your local disk:

    git clone git@github.com:YourLogin/benchopt.git
    cd benchopt
    
  1. Add the upstream remote. This saves a reference to the main benchopt repository, which you can use to keep your repository synchronized with the latest changes:

    git remote add upstream https://github.com/benchopt/benchopt
    
  2. Check that the upstream and origin remote aliases are configured correctly by running git remote -v which should display:

    origin  git@github.com:YourLogin/benchopt.git (fetch)
    origin  git@github.com:YourLogin/benchopt.git (push)
    upstream        https://github.com/benchopt/benchopt (fetch)
    upstream        https://github.com/benchopt/benchopt (push)
    

You should now have a working installation of benchopt, and your git repository properly configured. The next steps now describe the process of modifying code and submitting a PR:

  1. Synchronize your main branch with the upstream/main branch, more details on GitHub Docs:

    git switch main
    git fetch upstream
    git merge upstream/main
    
  2. Create a feature branch to hold your development changes:

    git switch -c my_feature
    

    and start making changes. Always use a feature branch. It’s good practice to never work on the main branch!

  3. Develop the feature on your feature branch on your computer, using Git to do the version control. When you’re done editing, add changed files using git add and then git commit:

    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
    
  4. Follow these instructions to create a pull request from your fork.

Note

It is often helpful to keep your local feature branch synchronized with the latest changes of the main benchopt repository:

git fetch upstream
git merge upstream/main

Documentation#

We are glad to accept any sort of documentation: function docstrings, reStructuredText documents (like this one), tutorials, etc. reStructuredText documents live in the source code repository under the doc/ directory.

You can edit the documentation using any text editor, and then generate the HTML output by typing, in a shell:

pip install benchopt[doc]
cd doc/
make html
firefox _build/html/index.html

Testing a benchmark#

To ensure that the benchmark is reproducible and can be run by others, benchopt provides a set of tools to test that the benchmark is properly formatted and that it can be installed and run easily. This page describes the various tweaks that can be made to the benchmark tests.

Basic philosophy#

The test run by benchopt test will make sure that:

  1. All classes can be retrieved easily from the benchmark, even when some dependencies are missing.

  2. The classes which have dependencies can be properly installed in a new environment and can then be imported.

  3. The datasets are compatible with the objective API.

  4. The solvers can all be run for a few number of iterations.

  5. For convex problems, the tests will also check that the solution is optimal for a small enough problem. This test can be deactivated by setting the Objective.is_convex flag to False.

Test for Solver run#

To ensure point 4, benchopt needs to load at least one small dataset that is compatible with each solver. This is why each benchmark needs to implement at least a Simulated dataset, that will be used for testing purposes. However, some solvers require different datasets and objective settings to be able to run. There are two ways to ensure that a solver can find an appropriate configuration:

  • In the simulated dataset, one can add the class attribute test_parameters, which stands for a list of parameters that will be tried to test the solver. For each solver, at least one of these configurations should be compatible (not skipped). See benchopt.github.io/how.html#example-of-parametrized-simulated-dataset

  • The solvers can also provide a test_config class attribute, which is a dictionary with optional keys dataset, objective. The value of these keys should be a dictionary of parameters for the classes Dataset and Objective, that will be compatible with the given Solver.

Test configuration#

In some cases, some tests should be skip, or xfail for a given benchmark. You can configure this for benchopt using a test_config.py file at the root of the benchmark. Implementing a function named check_TESTNAME with the same argument as the original test, you can then call pytest.xfail or pytest.skip to mark the test appropriately. For instance, to skip install tests for solver XXX, you can have the following:

# test_config.py
import pytest


def check_test_solver_install(solver_class):
    if solver_class.name.lower() == 'xxx':
        pytest.skip('XXX is not easy to install automatically.')