Skip to content

Latest commit

 

History

History
53 lines (36 loc) · 2.23 KB

CONTRIBUTING.md

File metadata and controls

53 lines (36 loc) · 2.23 KB

Contributing

Philosophy

AMD welcomes contributions 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.

Submitting a Pull Request

To contribute changes to rocSOLVER, open a pull request targeting the develop branch. Pull requests will be tested and reviewed by the AMD development team. AMD may request changes or modify the submission before acceptance.

Interface requirements

The public interface must be:

  • C99 compatible
  • Source and binary compatible with previous releases
  • Fully documented with Doxygen and Sphinx

All identifiers in the public headers must be prefixed with rocblas, ROCBLAS, rocsolver, or ROCSOLVER.

All user-visible symbols must be prefixed with rocblas or rocsolver.

Style guide

In general, follow the style of the surrounding code. All code is auto-formatted using clang-format. To apply the rocsolver formatting, run clang-format -i -style=file <files> on any files you've changed. You can install git hooks to do this automatically upon commit by running scripts/install-hooks --get-clang-format. If you find you'd rather not use the hooks, they can be removed using scripts/uninstall-hooks.

Tests

To run the rocSOLVER test suite, first build the rocSOLVER test client following the instructions in Building and Installation. Then, run the rocsolver-test binary. For a typical build, the test binary will be found at ./build/release/clients/staging/rocsolver-test.

The full test suite is quite large and may take a long time to complete, so passing the --gtest_filter=<pattern> option to rocsolver-test may be useful during development. A fast subset of tests can be run with --gtest_filter='checkin*', while the extended tests can be run with --gtest_filter='daily*'.

Rejected contributions

Unfortunately, sometimes a contribution cannot be accepted. The rationale for a decision may or may not be disclosed.