From 5c341503a7a6f9fbcab215db6378ba89cdb6b7d3 Mon Sep 17 00:00:00 2001 From: Wayne Franz Date: Fri, 15 Mar 2024 16:20:12 -0400 Subject: [PATCH] Add Contribution Readme (#347) * Add Contribution Readme Add a CONTRIBUTING.md file that contains information users will need if they want to contribute to the repository. This is based on the template located here: https://github.com/ROCm/.github/blob/main/docs/templates/contributing_template.md * Update CONTRIBUTING.md minor edits * Update CONTRIBUTING.md Missed a word. --------- Co-authored-by: randyh62 <42045079+randyh62@users.noreply.github.com> Co-authored-by: Young Hui - AMD <145490163+yhuiYH@users.noreply.github.com> --- CONTRIBUTING.md | 114 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 114 insertions(+) create mode 100644 CONTRIBUTING.md diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md new file mode 100644 index 00000000..d21bbd85 --- /dev/null +++ b/CONTRIBUTING.md @@ -0,0 +1,114 @@ + + + + + + +# Contributing to hipCUB # + +We welcome contributions to hipCUB. Please follow these details to help ensure your contributions will be successfully accepted. + +## Issue Discussion ## + +Please 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 ## + +The purpose of hipCUB is to provide a thin wrapper library on top of rocPRIM or CUB. This wrapper allows users to port CUB projects to HIP so that they can also be run on AMD hardware. + +Because it is a wrapper, the implementations of the algorithms that hipCUB API calls are not contained within hipCUB. +They exist within rocPRIM and CUB - the hipCUB API functions simply delegate the work to these underlying libraries. +This delegation should be performed in a manner that minimizes overhead. + +When a pull request is created, a number of automated checks are run. These checks: +- test the change on various OS platforms (Ubuntu, RHEL, etc.) +- run on different GPU architectures (MI-series, Radeon series cards, etc.) +- run benchmarks to check for performance degredation + +In order for change to be accepted: +- it must pass all of the automated checks +- it must undergo a code review + +The GitHub "Issues" tab may also be used to discuss ideas surrounding particular features or changes, before raising pull requests. + +## Code Structure ## + +hipCUB is a header-only library. Library code lives inside of /hipcub/include. +Code is organized by the level-of-scope at which it operates. For example, the following +subdirectories are organized by hardware-level scope: +* device/ contains headers for device-level algorithms +* block/ contains headers for block-level algorithms +* warp/ contains headers for warp/wavefront-level algorithms + +The following subdirectories are organized according to software-scope level: +* grid/ contains headers for grid-level operations (barriers, queues, etc.) +* thread/ contains headers for thread-level operations (load/store, scan, reduce, etc.) + +Finally, the iterator/ subdirectory provides access to the iterators that are used to interact with most algorithms in the library. + +Back at the root level, you can find tests and benchmarks located inside directories of the same name. +There is also an examples/ folder that contains a number of sample API use cases. + +## Coding Style ## + +C and C++ code should be formatted using `clang-format`. Use the clang-format version for Clang 9, which is available in the `/opt/rocm` directory. Please do not use your system's built-in `clang-format`, as this is an older version that will have different results. + +To format a file, use: + +``` +/opt/rocm/hcc/bin/clang-format -style=file -i +``` + +To format all files, run the following script in hipCUB 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 ## + +Our code contribution guidelines closely follows the model of [GitHub pull-requests](https://help.github.com/articles/using-pull-requests/). + +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. +Releases are cut to release/rocm-rel-x.y, where x and y refer to the release major and minor numbers. + +### Deliverables ### + +Code that introduces new features should have test coverage and benchmark coverage. +hipCUB tests are located in the test/hipcub/ directory, while benchmarks can be found in the benchmark/ directory. + +### Process ### + +After you create a PR, you can take a look at a diff of the changes you made using the PR's "Files" tab. + +PRs must pass through the checks and the code review described in the [Acceptance Criteria](#acceptance-criteria) section before they can be merged. + +Checks may take some time to complete. You can view their progress in the table near the bottom of the pull request page. You may also be able to use the links in the table +to view logs associated with a check if it fails. + +During code reviews, another developer will take a look through your proposed change. If any modifications are requested (or further discussion about anything is +needed), they may leave a comment. You can follow up and respond to the comment, and/or create comments of your own if you have questions or ideas. +When a modification request has been completed, the conversation thread about it will be marked as resolved. + +To update the code in your PR (eg. in response to a code review discussion), you can simply push another commit to the branch used in your pull request.