Skip to content

docs(composablekernel): fix 'seperate' typos in ck_tile README and CMake comment - #14465

Merged
illsilin merged 1 commit into
ROCm:developfrom
happy520ai:fix-composablekernel-typos
Oct 28, 2026
Merged

illsilin merged 1 commit into
ROCm:developfrom
happy520ai:fix-composablekernel-typos

Conversation

@happy520ai

Copy link
Copy Markdown
Contributor

Summary

Fix two occurrences of the misspelling seperate → separated:

  1. projects/composablekernel/example/ck_tile/01_fmha/README.md — the group-mode -s argument description said "comma seperated int to set per batch seqlen". The same document already uses the correct "comma-separated ints" two lines below (the -s_k description), so this also makes the wording consistent.
  2. projects/composablekernel/CMakeLists.txt — a comment above CK_PARALLEL_LINK_JOBS said "Seperate linking jobs from compiling".

Docs-only change, no code impact.

Reviewing

Purely textual; the two changed lines can be verified by eye.

…ake comment

Fix two occurrences of 'seperate' -> 'separated': the fmha group-mode
README argument description and a CMakeLists comment. The same README
already uses 'comma-separated ints' two lines below, so this also makes
the wording consistent.
@happy520ai
happy520ai requested review from a team as code owners October 27, 2026 14:37
@github-actions github-actions Bot added documentation project: composablekernel ck: attention-moe Used to tag composablekernel PRs that require approval from Attention/MOE review team. labels Oct 27, 2026
@therock-pr-bot

Copy link
Copy Markdown

❌ PR Check — Action Required

Check Status Details
📝 PR Description ❌ Fail Error: PR description must reference a JIRA ID, ISSUE ID, or a GitHub closing keyword.
Expected: include a JIRA ID / ISSUE ID line (separator : or -, or omitted; value may be a JIRA key, a number with/without #, or a link), OR a closing keyword + issue reference. Accepted examples:
• JIRA ID : TESTAUTO-6039
• JIRA ID - #330
• JIRA ID #330
• JIRA ID (on separate line)
ROCM-25757
• ISSUE ID : TESTUTO-3334
• ISSUE ID #3334
• ISSUE ID - TESTAUTO-3433
• ISSUE ID (on separate line)
AIRUNTIME-2352
• ISSUE ID : https://github.com/<org_name>/<repo_name>/issues/1234
• Closes #10
• Fixes octo-org/octo-repo#100
• Resolves: #123
• #123
• https://github.com/<org_name>/<repo_name>/issues/123
Current: no valid JIRA/ISSUE/closing-keyword reference found
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ⏳ Pending ⏳ Still running…
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —

⚠️ 1 policy check(s) failed. Please address the issues above before this PR can be Reviewed.

🚫 Please fix the failed policies

  • ❌ PR Description

The Not ready to Review label was added to this PR. Once all policies pass, the label is removed automatically.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@therock-pr-bot

Copy link
Copy Markdown

🚫 Please fix the failed policies before requesting reviews.

The following policy checks failed:

  • ❌ PR Description

The Not ready to Review label has been added to this PR.
Once all policies pass, the label will be removed automatically.

@assistant-librarian assistant-librarian Bot added the external contribution Code contribution from users community.. label Oct 27, 2026
@illsilin illsilin added the ci:skip Skip all CI builds/tests for this PR label Oct 27, 2026
@illsilin
illsilin enabled auto-merge (squash) October 27, 2026 15:54
@illsilin
illsilin merged commit 5bee927 into ROCm:develop Oct 28, 2026
49 of 56 checks passed
assistant-librarian Bot pushed a commit to ROCm/composable_kernel that referenced this pull request Oct 28, 2026
docs(composablekernel): fix 'seperate' typos in ck_tile
 README and CMake comment (#14465)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## Summary

Fix two occurrences of the misspelling `seperate` → `separated`:

1. `projects/composablekernel/example/ck_tile/01_fmha/README.md` — the
group-mode `-s` argument description said "comma seperated int to set
per batch seqlen". The same document already uses the correct
"comma-separated ints" two lines below (the `-s_k` description), so this
also makes the wording consistent.
2. `projects/composablekernel/CMakeLists.txt` — a comment above
`CK_PARALLEL_LINK_JOBS` said "Seperate linking jobs from compiling".

Docs-only change, no code impact.

## Reviewing

Purely textual; the two changed lines can be verified by eye.

Co-authored-by: happy520ai <210128019+happy520ai@users.noreply.github.com>
@assistant-librarian

Copy link
Copy Markdown
Contributor

TheRock Submodule Bump Activity

Newest first

shumway pushed a commit to ROCm/composable_kernel that referenced this pull request Nov 4, 2026
docs(composablekernel): fix 'seperate' typos in ck_tile
 README and CMake comment (#14465)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## Summary

Fix two occurrences of the misspelling `seperate` → `separated`:

1. `projects/composablekernel/example/ck_tile/01_fmha/README.md` — the
group-mode `-s` argument description said "comma seperated int to set
per batch seqlen". The same document already uses the correct
"comma-separated ints" two lines below (the `-s_k` description), so this
also makes the wording consistent.
2. `projects/composablekernel/CMakeLists.txt` — a comment above
`CK_PARALLEL_LINK_JOBS` said "Seperate linking jobs from compiling".

Docs-only change, no code impact.

## Reviewing

Purely textual; the two changed lines can be verified by eye.

Co-authored-by: happy520ai <210128019+happy520ai@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip Skip all CI builds/tests for this PR ck: attention-moe Used to tag composablekernel PRs that require approval from Attention/MOE review team. documentation external contribution Code contribution from users community.. Not ready to Review project: composablekernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants