Repository navigation
[DONE] 가이드 연습 벽 카드의 노브가 커뮤니티 카드와 같은 셈으로 돌고, Lab 주석이 포크 번호 대신 말로 적는다 - #246
Merged
Merged
Conversation
… controls.ts The practice wall copied the community card's old step, floored at 0.001 and rounded to three decimals, so a 0.0001..0.0004 knob fell to 0 on one notch. Also drops the fork PR number from two Lab test comments and counts the Text panel in the Lab shell's header. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015AhWQdepAbAamwZVKFFLfz
patternflow-bot
marked this pull request as ready for review
October 25, 2026 05:31
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 25, 2026
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 25, 2026
This was referenced Oct 26, 2026
This was referenced Oct 28, 2026
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
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.
lsh / https://claude.ai/code/session_015AhWQdepAbAamwZVKFFLfz
가이드의 연습 벽 카드가 커뮤니티 카드의 옛 노브 셈을 베껴 두어서, 0.0001..0.0004 처럼 좁은 노브는 한 칸 돌리면 0 으로 떨어지고 막대가 가운데를 15% 로 그렸어요. 이제 두 카드가
controls.ts의 한 함수로 돌고, 가이드 시뮬레이터와 첫 화면 3D 패널도 노브의 0..1 을 같은 곳에서 읽어요. Lab 시험 주석 둘의 포크 번호 「#184」 는 말로 바꾸고, Lab 셸 머리 주석은 Text 를 넣어 패널 아홉이라고 해요.좋아지는 것: 없음
할 일
deviceSim,LedMatrixTexture가controls.ts를 읽게PatternLabClient.tsx머리 주석의 패널 수check:voice88f9663: Drift checks, Web CI)053e687그대로라 merge 할 것 없음)한 일
①
CodePanel.test.tsx:213:214와PixelPanel.test.tsx:62:63의 「(#184)」 를 「before the Lab had text layers」 로 바꿨어요.app/pattern-lab/,lib/lab/에서 주석 속#숫자를 다시 찾으니 이 둘뿐이었어요.②
lib/pattern/controls.ts에knobWheelValue(value, range, direction)을 더했어요. 범위의 25분의 1 을 한 칸으로 움직이고,tidyKnobValue로 소수 끝을 다듬고, 범위 안에 잡아 둬요. 커뮤니티 카드(components/community/PatternCard.tsx의handleWheel)가 쓰던 셈 그대로예요. 그 카드와 연습 벽 카드(components/guide/desk/community/parts.tsx의turnKnob)가 이 함수를 불러요. 연습 벽 카드의 막대는normalizeKnob으로 읽어요.lib/guide/deviceSim.ts의knobNormalized와components/3d/LedMatrixTexture.ts의getLogicalKnobNormalized도normalizeKnob을 불러요. 둘은 이미 0.0001 로 나눠서 값은 그대로예요. 이제KNOB_SPAN_FLOOR를 읽는 파일이controls.ts뿐이라export를 뗐어요.③
PatternLabClient.tsx:6~:8이 「nine dockable panels」 이고 목록에 Text 가panels/registry.tsx차례대로 들어가요.증거
새
components/guide/desk/community/parts.test.tsx는 연습 벽 카드를 그려서 ArrowUp, ArrowDown 으로 K1 을 돌리고 막대 폭을 읽어요. dev 의parts.tsx로 돌린 결과예요.이 브랜치에서는
controls.test.ts에 더한 넷까지 두 파일 32개가 다 지나요(Tests 32 passed (32)).연습 벽의 네 패턴(Origin, Pattern 1~3)은
@knobs가 없어서 기본 범위[0,1],[0.1,10],[0,4.9],[0,1]이에요. 한 칸이 0.04, 0.396, 0.196, 0.04 라 세 자리 반올림이 아무것도 바꾸지 않아요. 네 패턴의 노브 넷을 가운데서 위아래로 40칸씩, 옛 셈과 새 셈으로 나란히 돌리니 1,280칸 가운데 다른 값이 0개였어요. 그래서 가이드 화면은 그대로라 사진은 찍지 않았어요.다섯 검사(
web/)의 끝줄이에요.check:ci의virtual-panel과 다섯 벡터 묶음은deviceSim을 바꾼 뒤에도 다 지나요.PR 검사는
88f9663에서 「Drift checks | completed | success」, 「Web CI | completed | success」 예요.실행법
cd web npx vitest run src/components/guide/desk/community/parts.test.tsx src/lib/pattern/controls.test.ts가정
controls.ts를 읽게 하라고 했어요. 연습 벽 카드가 상수 셋만 가져다 쓰면 25분의 1, 다듬기, 범위 잡기를 또 한 벌 적게 돼서, 커뮤니티 카드와 같이 부를 함수 하나로 모았어요. 그래서 지시에 없던components/community/PatternCard.tsx도 고쳤어요. 그 카드의 셈은 바뀌지 않아요.components/guide/desk/community/CommunityApp.tsx:109의toFixed(3)은 튜토리얼이 K1 을 0.04 씩 열 번 돌리는 연출이라 두었어요.pages.tsx:292의toFixed(3)은 값을 세 자리로 보이는 글이고PatternDetailClient.tsx:346과 같아요.components/3d/caseModels.test.ts:125주석에 「PR [DONE] web: the build page's 3D preview becomes the case you pick — official v3.9, Besoiobiy, SimonePDA #3's detail review」 가 있어요. Lab 코드가 아니고 포크 PR 인지 원래 레포 PR 인지 가릴 수 없어 두었어요.Changelog
Web
knobWheelValueinlib/pattern/controls.ts, and the guide's device simulator and the first screen's 3D panel read a knob's 0..1 throughnormalizeKnob.바퀴
바퀴 97 · 지시 7 「주석 · 사본 맞추기」
경험
언제: 가이드 연습 벽(
components/guide/desk/community/)의 카드를 vitest 로 그려 시험할 때교훈:
knobsOf는PATTERNS에 없는 id 에[0.5, 0.5, 0.5, 0.5]를 돌려주니, 시험용 패턴을 쓰면usePractice.setState({ knobs: { <id>: … } })로 값을 먼저 심고 카드의data-pc-card에 ArrowUp keyDown 을 보내요근거: #246 에서 0.0001..0.0004 노브를 가진 시험 패턴으로 카드를 그렸어요. 값을 심고 나서야 카드의 셈만 잴 수 있었고, dev 카드에서 0, 0, 15% 로 빨갰어요
다음 바퀴
!단언 가운데 타입으로 좁힐 수 있는 일곱이 빠진다」:isValidMatrix를 부르는 곳은 아직legacyDraft.ts와serialize.ts둘뿐이에요.globals.css에서 아무도 쓰지 않는 옛 Inside 패널 규칙 377줄이 빠진다」: 10위와 같은 파일이라 8위를 먼저 하면 10위의 줄 번호가 앞당겨져요.https://claude.ai/code/session_015AhWQdepAbAamwZVKFFLfz
Generated by Claude Code