Skip to content

[Docs] Follow the backend example's tuning tests, and state the backends page without implementation-specific counts - #75

Merged
lcy-seso merged 4 commits into
tile-ai:mainfrom
lcy-seso:docs/backend-example-tuning
Oct 17, 2026
Merged

lcy-seso merged 4 commits into
tile-ai:mainfrom
lcy-seso:docs/backend-example-tuning

Conversation

@lcy-seso

@lcy-seso lcy-seso commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Problems

  • Follow TileOPs main: serving_target, built_entries, and a kernel's request_tune and config lcy-seso/tileops-backend-example#4 added tests/test_tuning.py; the backends page lists four test files and 24 passed.
  • The page states facts that go stale with the next change: the test count, the GPU it ran on, the dev image's tilelang tag, the number of test files, how many kernels GEMM declares, and the list of CUDA architecture markers.
  • The template step said only the op and target names need substituting; the tests also import the package and the kernel classes, create tensors on CPU and assert the CPU device type.
  • The link to the example's tests/ points at tree/main, which returns 404: the repository's branch is master.

Changes

  • The layout table lists tests/test_tuning.py, English and Chinese.
  • The test run says all tests pass, none skipped, with and without a GPU; the docker command takes a published <tag>, in the form the development guide's dev-image section gives.
  • The template step lists the three changes the tests need: the package, target, op and kernel class names; the device; and op- or kernel-specific assertions. It links tree/master/tests. GEMM "declares several" kernels.
  • The marker table names architecture markers by their form, sm<NN>, and counts three kinds of marker.
  • The Chinese page is reworded as technical prose; the template steps are statements.

…on master

The example now defines request_tune() and config on its kernels and checks them in
tests/test_tuning.py (lcy-seso/tileops-backend-example#4), so the layout table lists that
file, the test run reports 28 passed, and the template steps carry over five test files.
The link to the example's tests named a main branch the repository does not have; it
names master.
Copilot AI balanced review requested due to automatic review settings October 17, 2026 16:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

… the implementation

The page gave the test count, the GPU it was run on, the dev image's tilelang tag, the
number of test files and how many kernels GEMM declares; each goes stale with the next
change. It now says the tests all pass with and without a GPU, takes <tag> from the
development guide, and leaves the counts out. The template step names every name the
tests depend on: package, target, op and kernel class.

The Chinese page also drops colloquial phrasing and spaces Latin terms from Chinese text.
@lcy-seso lcy-seso changed the title [Docs] Follow the backend example's tuning tests, and link its tests on master [Docs] Follow the backend example's tuning tests, and state the backends page without implementation-specific counts Oct 17, 2026
… pick the dev-image tag

The template step listed only names to substitute; the tests also create tensors on CPU
and assert the CPU device type, and some assertions are specific to an op or kernel. The
dev-image guide gives the tag's form, not a current tag, so the page asks for a published
tag in that form.
…ileOPs has today

The marker table listed sm80 to sm90 and called them, with two other markers, three
markers. A new architecture marker would leave the list incomplete; the table now names
the form sm<NN> and three kinds of marker. The Chinese page also spaces a link from the
text that follows it, as elsewhere on the page.
@lcy-seso
lcy-seso merged commit 6eafd37 into tile-ai:main Oct 17, 2026
3 checks passed
@lcy-seso
lcy-seso deleted the docs/backend-example-tuning branch October 17, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants