Skip to content

스트리밍 인코더 - #10

Merged
owjs3901 merged 3 commits into
dev-five-git:mainfrom
youknowone:streaming-encoder
Jun 12, 2025
Merged

스트리밍 인코더#10
owjs3901 merged 3 commits into
dev-five-git:mainfrom
youknowone:streaming-encoder

Conversation

@youknowone

Copy link
Copy Markdown
Contributor

입력을 스트리밍해서 넣으면 결과도 스트리밍해서 받을수 있도록 합니다.

긴 글 처리에 유용할수도 있고, decoder를 만들 때 테스트를 쉽게 작성하려면 컨텍스트를 encode에서 분리시키는 편이 유리합니다.

@changeset-bot

changeset-bot Bot commented Jun 12, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 69de951

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov

codecov Bot commented Jun 12, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 95.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
libs/braillify/src/lib.rs 95.00% 3 Missing ⚠️
Files with missing lines Coverage Δ
libs/braillify/src/lib.rs 92.63% <95.00%> (-0.36%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread libs/braillify/src/lib.rs
@owjs3901

Copy link
Copy Markdown
Contributor

기본적으로 스트리밍이 무엇인지는 아는데, rust에서도 그것이 가능한지 몰랐습니다
다만 구현 코드를 보면 제가 알고 있는 스트리밍이랑 개념이 차이가 조금 있는 것 같습니다 (제가 러스트 프로젝트가 겨우 2번째라서 보고도 인지를 못하고 있을 가능성이 더 큽니다)

혹시 설명이 가능하실까요?

@youknowone

youknowone commented Jun 12, 2025

Copy link
Copy Markdown
Contributor Author

추가된 테스트가 이 기능을 (부적절한 예제로) 테스트하고 있는데요 https://github.com/dev-five-git/braillify/pull/10/files#diff-fa1337336c3bbeb7e63aa754a905f34b2f6008043ef72a7aba76c70cbab55b84R784-R791

제가 점자를 잘 몰라서 올바른 예시를 들어서 설명은 어렵지만, 이를테면 긴 문자열 "AB" 가 있을 때, encode("AB")와 encode("A") + encode("B") 의 결과는 중간에 삽입되는 추가문자로 인해서 다를 수 있습니다.

Encoder를 쓰면 encoder.encode("A") 를 하고 나면 상태를 그대로 보존하고 있기 때문에 encoder.encode("B") 를 했을 때 "A" 에 의해서 추가되어야 하는 문자가 있으면 올바르게 이어 붙여서 인풋을 분리해서 넣어도 "AB" 를 한번에 넣었을때와 동일한 결과를 얻게 됩니다.

Comment thread libs/braillify/src/lib.rs
@owjs3901
owjs3901 merged commit e1b62d4 into dev-five-git:main Jun 12, 2025
jessieyeon added a commit to jessieyeon/braillify that referenced this pull request Aug 7, 2026
…ve-git#10 dev-five-git#11)

- 전역 box-sizing:border-box 로 padding 이 flex-basis 를 어긋내지 않으므로
  바깥 Flex 래퍼를 제거하고, 대신 minW=0 으로 콘텐츠 최소 폭 확장을 막는다.
- 안쪽 Flex flexDirection=column 을 VStack 으로 교체.
- oxlint 지적(prop 정렬, no-namespace) 정리.
jessieyeon added a commit to jessieyeon/braillify that referenced this pull request Aug 7, 2026
…-git#9 dev-five-git#10)

- dev-five-git#5 탭 borderRadius 를 50% 로 변경
- dev-five-git#6 수학 데모의 입출력부를 MathTransPanel 클라이언트 컴포넌트로 분리,
  MathTrans 는 헤딩만 남기고 use client 제거
- dev-five-git#9 dev-five-git#10 MathTransInput 의 flex=1 두 개 제거, 좌우 폭 분배는 호출부
  (MathTransPanel 의 행)에서 selectors 로 지정
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