Skip to content

fix: 동·호 중복 상세를 한 번만 유지 - #23

Open
jinon86 wants to merge 2 commits into
mainfrom
fix/22-dedupe-unit-tokens
Open

fix: 동·호 중복 상세를 한 번만 유지#23
jinon86 wants to merge 2 commits into
mainfrom
fix/22-dedupe-unit-tokens

Conversation

@jinon86

@jinon86 jinon86 commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

요약

  • 공백으로 분리된 동·호 토큰을 정제 마지막 단계에서 비교해 같은 단위는 첫 표기만 유지합니다.
  • 접두, 한글/영문 A~D 동 표기, 영문 대소문자와 앞자리 0이 다른 호수 표기를 비교용 키에서만 정규화합니다.
  • 첫 표기의 원래 문자열과 중복 사이의 건물명 등 다른 상세정보는 그대로 보존합니다.

오탐 방지

  • 제기동/기동은 서로 다른 법정동으로 유지합니다.
  • 영등포동2가, 테스트동아파트처럼 다른 토큰에 포함된 은 대상에서 제외합니다.
  • 외부 API 호출·판정과 운영 데이터 형식은 변경하지 않습니다.

검증

  • pytest -q --cov=address_cleaner --cov-report=term-missing --cov-fail-under=88
  • ruff check .
  • ruff format --check .
  • mypy
  • 테스트 결과: 195 passed, 4 subtests passed, coverage 93.54%

실제 주소·업로드 엑셀·API 키는 커밋하지 않았고 회귀 테스트는 모두 익명화했습니다.

Closes #22

@jinon86
jinon86 requested a review from seoseo-ai August 26, 2026 06:17
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