Contributing to rocSPARSE

Contributing to rocSPARSE#

AMD welcomes contributions to rocSPARSE from the community. Whether those contributions are bug reports, bug fixes, documentation additions, performance notes, or other improvements, we value collaboration with our users. We can build better solutions together. Follow these details to help ensure your contributions will be successfully accepted.

Our code contriubtion guidelines closely follow the model of GitHub pull-requests. This repository follows the git-flow workflow, which dictates a /master branch where releases are cut, and a /develop branch which serves as an integration branch for new code.

Issue discussion#

Use the GitHub Issues tab to notify us of issues.

  • Use your best judgement for issue creation. If your issue is already listed, upvote the issue and comment or post to provide additional details, such as how you reproduced this issue.

  • If you’re not sure if your issue is the same, err on the side of caution and file your issue. You can add a comment to include the issue number (and link) for the similar issue. If we evaluate your issue as being the same as the existing issue, we’ll close the duplicate.

  • If your issue doesn’t exist, use the issue template to file a new issue. * When filing an issue, be sure to provide as much information as possible, including script output so we can collect information about your configuration. This helps reduce the time required to reproduce your issue. * Check your issue regularly, as we may require additional information to successfully reproduce the issue.

  • You may also open an issue to ask questions to the maintainers about whether a proposed change meets the acceptance criteria, or to discuss an idea pertaining to the library.

Acceptance criteria#

rocSPARSE is a library that contains basic linear algebra subroutines for sparse matrices and vectors written in HIP for GPU devices. It is designed to be used from C and C++ code. The functionality of rocSPARSE is organized in the following categories:

In rocSPARSE we are interested in contributions that:

  • Fix bugs, improve documentation, enhance testing, reduce complexity.

  • Improve the performance of existing routines.

  • Add missing functionality found in one of the categories above.

  • Add additional sparse matrix formats or allow an existing format to be used with an existing routine.

Some of the routines in rocSPARSE allow users to choose between multiple different algorithms. This is useful for obtaining the most performance for different use cases (as some algorithms may perform better than others depending on the sparse matrix) or for satisfying important user requirements (such as run-to-run reproducibility). The following is a non-exhaustive list of reasons for including alternate algorithms:

  • Some algorithms may perform better when a sparse matrix has roughly the same number of non-zeros per row.

  • Some algorithms may perform better when a sparse matrix has a large variation in the number of non-zeros per row.

  • Some algorithms may perform better if they are allowed to use a large amount of device memory.

  • Some algorithms may perform better or worse depending on whether a user intends to perform the computation only once or many times.

  • Some algorithms may exist to allow for reproducibility between runs, for example by not using atomic operations.

  • Some algorithms may exist because they do not require any additional memory allocation or analysis phase.

  • Some algorithms may handle different ranges in sparse matrix size, i.e number of rows or number of non-zeros.

An opportunity exists here for contributors to add different algorithms that optimize for important user requirements and performance considerations. We encourage contributors to leverage the GitHub “Issues” tab to discuss possible additions they would like to add.

Exceptions#

rocSPARSE places a heavy emphasis on being high performance. Because of this, contributions that add new routines (or that modify existing routines) must do so from the perspective that they offer high performance in relation to the hardware they are run on. Typically in rocSPARSE this evaluation is done using approximations of GFLOPS/s or GB/s and comparing this to what the device is estimated to achieve. Comparison to other sparse math libraries is also useful.

Additionally, when adding new routines, these routines must offer enough value to enough users to be deemed worth including. Because compile times, binary sizes, and general library complexity are important considerations, we reserve the right to make decisions on whether a proposed routine is too niche or specialized to be worth including.

Code structure#

The following is the structure of the rocSPARSE library in the GitHub repository. A more detailed description of the directory structure can be found in the rocSPARSE documentation.

The library/include/ directory contains the rocsparse.h header (which itself includes headers defining the public API of rocSPARSE). The library/include/ directory also contains the headers for all the rocSPARSE public types.

The library/src/ directory contains the implementations of all the rocSPARSE routines. These implementations are broken up into directories describing the category the routine belongs too, i.e. level1, level2, level3, etc. These directories contain both the C++ and HIP kernel code.

The clients/ directory contains the testing and benchmarking code as well as all the samples demonstrating rocSPARSE usage.

The docs/ directory contains all of the documentation files.

The scripts/ directory contains potentially useful python and shell scripts for downloading test matrices (see scripts/performance/matrices/) as well as plotting tools. See rocSPARSE documentation for more details.

Coding style#

In general, follow the style of the surrounding code. C and C++ code is formatted using clang-format. Use the clang-format version installed with ROCm (found in the /opt/rocm/llvm/bin directory). Do not use your system’s built-in clang-format, as this is a different version that may result in incorrect results.

To format a file, use:

` /opt/rocm/hcc/bin/clang-format -style=file -i <path-to-source-file> `

To format all files, run the following script in rocSPARSE directory:

` #!/bin/bash git ls-files -z *.cc *.cpp *.h *.hpp *.cl *.h.in *.hpp.in *.cpp.in | xargs -0 /opt/rocm/hcc/bin/clang-format  -style=file -i `

Also, githooks can be installed to format the code per-commit:

` ./.githooks/install `

Pull request guidelines#

When you create a pull request, you should target the default branch. Our current default branch is the develop branch, which serves as our integration branch.

Deliverables#

When raising a PR in rocSPARSE here are some important things to include:

  1. For each new file in the repository, include the licensing header

Listing 4 rocsparse_file_header#
/* ************************************************************************
* Copyright (C) 20xx Advanced Micro Devices, Inc. All rights Reserved.
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
* THE SOFTWARE.
*
* ************************************************************************ */

and adjust the date to the current year. When simply modifying a file, the date should automatically be updated when using the pre-commit script.

  1. When adding a new routine, make sure you are also adding appropriate testing code. These new unit tests should integrate within the existing GoogleTest framework. This typically involves adding the following files:

  • testing_<routine_name>.cpp file in the directory clients/testing/

  • test_<routine_name>.cpp file in directory clients/tests/

  • test_<routine_name>.yaml file in directory clients/tests/

See existing tests for guidance when adding your own.

  1. When modifiying an existing routine, add appropriate testing to test_<routine_name>.yaml file in directory clients/tests/.

  2. Tests must have good code coverage.

  3. At a minimum, rocSPARSE supports the following data/compute formats:

  • float

  • double

  • rocsparse_float_complex

  • rocsparse_double_complex

So when adding a new routine that uses data/compute values, support at least these four types.

  1. Ensure code builds successfully. This includes making sure that the code can compile, that the code is properly formatted, and that all tests pass.

  2. Do not break existing test cases

Process#

When a PR is raised targetting the develop branch in rocSPARSE, CI will be automatically triggered. This will:

  • Test that the PR passes static analysis (i.e ensure clang formatting rules have been followed).

  • Test that the documentation can be properly built

  • Ensure that the PR compiles on different OS and GPU device architecture combinations

  • Ensure that all tests pass on different OS and GPU device architecture combinations

Feel free to ask questions on your PR regarding any CI failures you encounter.

  • Reviewers are listed in the CODEOWNERS file

Contents