Skip to content

IM3-263: fix(map): 자사 매장 별표 표시 fix — '기타' 카테고리 통과 + 크기 축소#208

Merged
yezin013 merged 2 commits into
devfrom
IM3-263-ai-summary-layout
May 7, 2026
Merged

IM3-263: fix(map): 자사 매장 별표 표시 fix — '기타' 카테고리 통과 + 크기 축소#208
yezin013 merged 2 commits into
devfrom
IM3-263-ai-summary-layout

Conversation

@yezin013

@yezin013 yezin013 commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • 자사 매장 별표가 안 뜨던 회귀 fix — kakao_store.category="기타" (분류 누락) 매장도
    통과시킴
  • 별표 마커 크기 통일 18×18px — 이전 디버그용 36px 확대 정리

변경

backend/src/main.py (_collect_same_brand_locations)

옵션 A(브랜드-업종 misalign 차단)는 유지하되, 카카오 분류 누락 케이스("기타")는 통과:

  • 발견: 한신포차 시뮬에서 target_cat="한식음식점" 인데 한신포차 홍대점 매장의
    category="기타" 라 cat drop → 별표 0개
  • 룰 변경: store_cat != target_category AND store_cat != "기타" 일 때만 drop
  • brand 매칭은 이미 정확히 됐으니 분류 누락 매장도 자사 매장으로 인정

frontend MarketMap.tsx

  • Layer 2 (competitors 안 자사 브랜드 매칭): 24×24px → 18×18px
  • Layer 3 (sameBrandLocations): 36×36px (디버그 확대) → 18×18px
  • 두 별표 동일 사이즈로 통일

Test plan

  • 로그 확인: [same_brand] cat drop 0 으로 변함
  • 별표 18×18 사이즈 시각 확인 (frontend HMR)
  • 옵션 A misalign 차단 의도 보존 (메가커피 ↔ 치킨 같은 명확한 misalign 은 여전히 차단)
  • 리뷰어 검토

yezin013 and others added 2 commits May 7, 2026 12:10
backend/src/main.py (_collect_same_brand_locations):
- 옵션 A 카테고리 매칭 룰 완화 — kakao_store.category="기타" (분류 누락) 매장 통과
- 한신포차 같이 brand 매칭은 정확하지만 카테고리 미지정 케이스 해결
- misalign 차단(메가커피 vs 치킨 시뮬) 의도는 유지

frontend MarketMap.tsx:
- 별표 마커 크기 통일 18×18px (font 10) — 이전 디버그 확대(36px) 정리
- Layer 2 (competitors 자사 매칭) + Layer 3 (sameBrandLocations) 동일 사이즈

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@yezin013 yezin013 merged commit 5412d81 into dev May 7, 2026
1 check passed
@github-actions github-actions Bot changed the title fix(map): 자사 매장 별표 표시 fix — '기타' 카테고리 통과 + 크기 축소 IM3-263: fix(map): 자사 매장 별표 표시 fix — '기타' 카테고리 통과 + 크기 축소 May 7, 2026
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.

1 participant