Korean translation sync for PRs #225 → #245#246
Open
GzuPark wants to merge 6 commits intomodular:mainfrom
Open
Korean translation sync for PRs #225 → #245#246GzuPark wants to merge 6 commits intomodular:mainfrom
GzuPark wants to merge 6 commits intomodular:mainfrom
Conversation
- puzzle_09/essentials.md: translate LLDB pending-breakpoint explanation added in modular#229 - puzzle_23/gpu-thread-vs-simd.md: adopt `Index(idx)` load/store signatures from modular#226 - puzzle_29/memory_barrier.md: apply explicit `Float32` cast from modular#237 - update i18n-source-commit markers to the matching upstream commits so outdated-translation warnings clear
Resolve partial-sync gaps where modular#238's mechanical replacement landed but later upstream edits (modular#226, modular#235, modular#237, modular#244, modular#245) and missing concept paragraphs were left behind. - puzzle_09/second_case.md: apply modular#245 debugger transcript updates (line numbers, `var`/`def` keywords, `input`→`a` parameter rename) - puzzle_10/memcheck.md: translate modular#244 segfault warning block, new callout, and expanded compatibility note - puzzle_11/puzzle_11.md: translate the modular#238 pooling and sliding window concept paragraph (previously dropped) and the modular#243 single-block assumption blockquote - puzzle_23/elementwise.md, tile.md, vectorize.md: adopt `Index(...)` load/store signatures and `aligned_load` from modular#226/modular#235 - puzzle_26/warp_shuffle_xor.md: wrap 6 `shuffle_xor` offsets with explicit `UInt32(...)` per modular#237 - puzzle_27/block_prefix_sum.md: apply `Float32(num_bins)` and `Int32(belongs_to_target)` casts from modular#237 - update i18n-source-commit markers so outdated-translation warnings clear
The modular#238 TileTensor migration added educational concept paragraphs to the English Overview sections, but the Korean pass dropped them instead of translating. Restore the missing content so readers get the same intuition before the implementation. - puzzle_05/puzzle_05.md: restore the broadcasting definition paragraph (regression introduced by the mechanical rename), add the missing "boundary conditions" learning bullet, and expand the Broadcasting/Access patterns bullets to match upstream - puzzle_08/puzzle_08.md: translate the shared-memory introduction paragraph (on-chip storage, block locality, barrier coordination) - puzzle_12/puzzle_12.md: translate the parallel-reduction introduction paragraph (O(log n) tree reduction with barrier) - bump i18n-source-commit markers to 19dfa37 so the outdated translation banners clear
Tied off the remaining priority-3 files after a structural audit against the English sources. Most were already content-aligned — only three had real gaps; the rest just needed their source-commit markers refreshed so the "outdated translation" banners clear. Content additions: - puzzle_06/puzzle_06.md: translate the thread-block introduction paragraph that anchors global-index math for the rest of Part I - puzzle_26/puzzle_26.md: wrap the butterfly `shuffle_xor` offset with `UInt32(...)` per modular#237 (one call missed in the earlier pass) - puzzle_26/warp_prefix_sum.md: same `UInt32(offset)` cast for the warp-level total reduction Marker-only refreshes (40 files spanning puzzle_01 through puzzle_34): verified English and Korean cumulative diffs match token-by-token (line-count deltas come from translator-added blank lines in blockquotes, not content gaps), so only the header stamp moves forward.
SUMMARY, introduction, and howto were already translated to match the upstream changes (modular#232 discord invite link and modular#238 TileTensor rename / raw+layout_tensor sub-entry removals); only the stale source-commit markers were still triggering the "outdated translation" banner. - SUMMARY.md → 19dfa37 (modular#238 final TileTensor state) - introduction.md → 19dfa37 (modular#238 + modular#232 discord link) - howto.md → 3e0e980 (modular#232 discord link)
The link to `../puzzle_08/tile_tensor.md` was broken because that file was removed in commit 924961c (Simplify problems with two approaches), which merged the content into `puzzle_08/puzzle_08.md` but missed updating this cross-reference. Fixed in both English and Korean.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR syncs the Korean translation with upstream changes made to the English source after PR #225 was merged. The main upstream PRs reflected here are:
load/store→Index(idx))Key changes
Terminology migration (#238)
LayoutTensor→TileTensoracross all touched filesLayout.row_major(...)→row_major[...]()syntaxLayoutTensor[...].stack_allocation()→stack_allocation[dtype=..., address_space=...](row_major[...]())layout_tensor_preview.md→tile_tensor_preview.mdintroduction_layout_tensor.md→introduction_tile_tensor.mdlayout_tensor.md→tile_tensor.mdConcept paragraphs restored or added
These are #238 regressions and gaps where the mechanical term replacement dropped educational paragraphs from the Overview section.
puzzle_05/puzzle_05.md: restored broadcasting concept paragraph (regression)puzzle_06/puzzle_06.md: added thread-block concept paragraphpuzzle_08/puzzle_08.md: added shared-memory concept paragraphpuzzle_11/puzzle_11.md: added pooling / sliding window concept paragraphpuzzle_12/puzzle_12.md: added parallel reduction concept paragraphPer-PR edits
puzzle_09/essentials.md: pending breakpoint explanation, 12 lines (update: clarify LLDB breakpoint behavior with Mojo JIT compilation #229)puzzle_09/second_case.md: CUDA-GDB transcript line numbers,fn→def,input→a,varkeyword additions (Remove stale reference to LayoutTensor code #245)puzzle_10/memcheck.md: compute-sanitizer WARNING lines + segfault note + expanded compatibility note (Clarify compute-sanitizer warnings #244)puzzle_11/puzzle_11.md: single-block assumption block + multi-block halo code example (Specify single-block limitation on puzzle 11 pattern #243)puzzle_23/*:load/store[simd_width](Index(idx)),aligned_load,fn→def(Updates for puzzle 23 #226/Mdoc/fixes #235)puzzle_26/warp_shuffle_xor.md,warp_prefix_sum.md,puzzle_26.md:UInt32(offset)explicit casts (Update all implicit type casts to be explicit #237)puzzle_27/block_prefix_sum.md:Float32(num_bins),Int32(belongs_to_target)casts (Update all implicit type casts to be explicit #237)puzzle_27/puzzle_27.md: stride pseudocodefor→whilecorrection (Correct stride pseudocode in p27 #240)puzzle_29/memory_barrier.md:stencil_sum / Float32(stencil_count)(Update all implicit type casts to be explicit #237)SUMMARY.md,introduction.md,howto.md: Discord link update (Replace discord channel links with discord invite links #232) + TileTensor terminologyDrive-by fix
puzzle_10/puzzle_10.md(English and Korean): fixed broken cross-reference../puzzle_08/tile_tensor.md→../puzzle_08/puzzle_08.md. The target file was removed in commit 924961c (Simplify problems with two approaches), but this link was missed.Notes
puzzle_28/puzzle_28.mdthecopy_dram_to_sram_asyncdoc link keeps the/layout_tensor/path because the English source does the same at the tracked commit.