Contributing Guide#
Welcome to the Tensile project! If you’re thinking about contributing, this document is for you. We encourage you to read this guide to understand how to contribute to the project to ensure that your contributions are accepted and merged in a timely manner.
See also
If you haven’t already, please review Getting Started for an introduction to the project. For details on environment setup and day-to-day development processes, please refer to the Developer Guide.
Tensile’s development practice is based on the Gitflow Workflow. The develop branch is the default branch for development and is where all new features and bug fixes should be merged. After a PR is merged into develop, it will undergo extended testing and profiling. Pending all of these checks pass, it may be promoted to staging be included in the next release. If you would like to see the changes in the next release, please ensure that the PR is merged before the release branch is cut.
How to submit a Pull Request#
When making changes:
Create a fork of Tensile—please do not create feature branches directly in ROCm/Tensile.
Clone your fork locally and set up your Development environment.
Create a feature branch off of develop and make changes to the code.
Issue
tox run -m precommit
and ensure that all checks pass.Commit you changes using the convention for Commit messages.
If you are updating documentation, issue
tox run -e docs
and verify the styling and formatting is what you expect.Push the changes to your fork.
Tip
Keeping the scope of new PRs as narrow as possible improves the chances it will be accepted. If you are making multiple changes, consider breaking them into separate PRs. Keeping PRs small supports timely code reviews, traceability, and straightforward reversions.
When opening a PR:
Ensure that your develop branch is up-to-date with the upstream develop branch—this may require a rebase or a merge.
Verify that your changes pass static analysis checks and all pre-checkin, host library, and unit tests by running
tox run -m prepr
—then go get a coffee, this could take up to an hour.Create the PRs against the ROCm/Tensile develop branch.
Fill in as many details as possible. Include a description, outcomes, notable changes, and environment information. This more information, the more likely the PR will be reviewed and merged in a timely manner.
Title the PR in present imperative tense, e.g., “Update kernel parameters” not “Updates” nor “Updated”.
Tip
If you need to merge develop into your feature branch after a PR is opened, use a merge instead of a rebase.
In general, refrain from force pushing once a feature branch is in PR as it is prone to gotchas in our CI system. Ideally, the git history is linear and clean before a PR is created. As such we encourage contributors to conduct any rebases or amends prior to opening a PR.
Once all checks pass and the PR is approved:
Ensure the title of the PR properly describes the changes, update if necessary.
Squash and merge the PR—if you are not a maintainer, a maintainer will do this for you. When merging a large change, use bullet points in the commit message to break down the changes.
Labels#
Label |
Effect |
---|---|
ci:profiling |
Adds the profiling job to the CI pipeline. Profiling artifacts will be saved for 10 days. |
ci:docs-only |
Only runs the docs/readthedocs job; omits all other pipeline jobs. |
Conventions and style guide#
General conventions#
Always use space indentation (4 spaces)—never commit a tab, e.g.,
\t
.
Python doc-strings#
Tensile uses autodoc to pull in documentation from doc-strings and integrate them into this site. Please use the following guidelines when writing Python functions and modules to maintain quality and consistency.
The all parameters and returned values should be identified with type-hints.
All functions should have a doc-string describing the parameters, return value, and any exception; however, if the function is small and the implementation is straightforward, a one-line doc-string is sufficient.
Do not include types directly in the doc-string, these should be added as type-hints in the function definition.
For doc-string styling, use the Google Python Style Guide.
Commit messages#
Use conventional commits.
Use the present imperative tense, e.g., “add” not “adds” nor “added”.
Don’t add a period (
.
) to the end of the message.