fix: exam_eng.hwp 9→8 페이지 회귀 해소 (#345)#346
Closed
planet6897 wants to merge 1 commit intoedwardkim:develfrom
Closed
fix: exam_eng.hwp 9→8 페이지 회귀 해소 (#345)#346planet6897 wants to merge 1 commit intoedwardkim:develfrom
planet6897 wants to merge 1 commit intoedwardkim:develfrom
Conversation
…#345 PR edwardkim#343 squash (c03fe32) 가 cumulative height advance 를 height_for_fit (= total - trail_ls) 로 변경하여 페이지당 더 많은 문단이 들어감. exam_eng.hwp 9→8 페이지 회귀 발생. 수정: - typeset_paragraph 의 advance 를 total_height 로 복원 (pre-edwardkim#343 동작). fit 판정은 height_for_fit 유지 (관대) — 마지막 문단의 trail_ls 가 페이지 끝에 걸쳐도 fit 인정. advance 만 total_height 로 정확히 누적해 다음 문단의 cumulative y 가 layout 의 vpos_end (= prev.vpos + lh + ls) 와 정합. - LAYOUT_DRIFT_SAFETY_PX (10px) 제거 — advance 정합으로 마진 불필요. - vpos-reset 기반 강제 column/page 분할 (cv==0 && pv>5000) 제거 — 21_언어 의 정상 분할을 +1 페이지로 부풀리는 부작용. 검증: - 21_언어 15, exam_math 20, exam_kor 24, KTX 27, aift 74, biz_plan 6 ✓ - exam_eng 8 → 10 (pre-edwardkim#343 baseline 9, 1 over — 회귀 해소 + 잔존) - cargo test 1000+ pass, 골든 변경 없음 수행계획서: mydocs/plans/task_m100_345.md
Owner
|
지금 devel 쪽을 main 으로 병합하는 작업을 진행중입니다. 끝나면 PR 리뷰 시작하겠습니다. |
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
2 participants
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.
Closes #345
원인
PR #343 의 squash 커밋 (`c03fe32`) 이 typeset 의 cumulative height advance 를 `fmt.total_height` → `fmt.height_for_fit` (= total - trailing_line_spacing) 로 변경. trail_ls 만큼 (~9.55px) 적게 누적되어 페이지당 문단이 더 들어감. exam_eng.hwp 9→8 페이지 회귀.
추가로 `LAYOUT_DRIFT_SAFETY_PX = 10` 마진 + vpos-reset 기반 강제 분할 (`cv==0 && pv>5000`) 도 같은 흐름으로 영향.
pre-#343 vs PR #343
수정
`src/renderer/typeset.rs::typeset_paragraph`:
검증
`cargo test --release` 1000+ pass, 골든 SVG 변경 없음.
잔존
exam_eng.hwp 가 본 수정 후 10 페이지 — pre-#343 baseline 9 대비 +1. 본 수정으로 회귀 (8) 는 해소되었으나 정확한 9 페이지로 복원되지는 않음. pre-#343 의 다른 미세 동작이 page 5 에서 한 번 더 split 을 트리거하는 것으로 추정.
→ 추가 디버깅이 필요하면 별도 후속 이슈로 분리 권고.
Test plan
🤖 Generated with Claude Code