Skip to content

plan(jgkim1008-daegu-mart-holiday): 대조군 정의·주 지표 수정 (데이터 수집 전) - #45

Merged
jsshin2022 merged 3 commits into
CausalInferenceLab:mainfrom
jgkim1008:jgkim1008/plan-amend
Oct 25, 2026
Merged

jsshin2022 merged 3 commits into
CausalInferenceLab:mainfrom
jgkim1008:jgkim1008/plan-amend

Conversation

@jgkim1008

Copy link
Copy Markdown
Contributor

무엇을 했나요?

#34 계획에서 두 가지를 고쳤습니다. 둘 다 판매액지수 값을 받기 전에 커밋했습니다. 관련 이슈: #30

  1. 대조군 정의 (2c7571e): "일요일 의무휴업을 유지한 광역시"를 "분석 기간 중 의무휴업일을 바꾸지 않은 광역시"로 바꿨습니다.
    • 대조 광역시의 휴무일 이력을 확인해 README에 표로 적었습니다.
    • 울산 남·북·중구는 2019~2023년 내내 둘째 수요일 + 넷째 일요일이었습니다. 웹 아카이브에 보관된 점포 휴점일과 언론 보도로 확인했습니다.
    • 기간 중 바뀌지 않아 울산은 유지하고, 울산을 뺀 추정을 반박 검정(drop_unit)으로 추가했습니다. 공통 실행기가 drop_unit을 아직 실행하지 않아 수동으로 돌립니다.
  2. 주 지표와 데이터 출처 (d822589): KOSIS 메타데이터만 조회해 통계표를 DT_1K41018(시도별 소매판매액지수, 분기)로 확정했습니다.
    • 시도 단위에는 '슈퍼마켓 및 잡화점'이 따로 없고 '슈퍼마켓· 잡화점 및 편의점'(A4)만 있어 주 지표를 이에 맞췄습니다.
    • 편의점 포함에 따른 희석을 한계에 추가했습니다.
  3. discussion.md > 계획에서 바뀐 것에 위 내용을 기록했습니다 (3bfad00).

케이스

cases/jgkim1008-daegu-mart-holiday

체크리스트

  • plan.yaml을 결과를 보기 전에 작성·커밋했습니다 (이후 변경 시 이유를 커밋 메시지와 discussion.md > 계획에서 바뀐 것에 기록)
  • 새 분석이면 계획만 담은 PR을 먼저 올렸습니다 (plan(jgkim1008-daegu-mart-holiday): 대구 대형마트 의무휴업 평일 전환 분석 계획 #34)
  • 결과 PR이면 discussion.md > 결과를 한 문장으로를 채웠습니다 — 결과 PR 아님
  • 데이터 출처와 라이선스를 plan.yaml > data_sources에 명시했습니다 (KOGL-1. 통계표 화면에서 최종 확인 예정)
  • API 키·.env·개인정보가 담긴 원자료·재배포 불가 데이터를 포함하지 않았습니다
  • 그림/수치는 fetch.py → make flow 실행으로 재현 가능합니다 — fetch.py는 다음 PR
  • make check (ruff + pytest)가 통과합니다
  • 다른 사람의 폴더나 core/를 (합의 없이) 수정하지 않았습니다
  • 원자료·가공 데이터 파일을 커밋하지 않았습니다

리뷰어에게

  • 공개 순서: 위 수정은 로컬에 커밋한 뒤, 이 PR을 올리기 전에 fetch.py로 데이터를 받았습니다. 수집할 때는 표 크기와 부산의 첫 3행만 출력해 형식을 확인했고, 대구와 비교하는 계산(make flow)은 하지 않았습니다. fetch.py는 이 PR이 합쳐진 뒤 별도 PR로 올립니다.
  • 울산 유지가 타당한지 봐 주세요. 기간 내내 같은 상태라 지역 고정효과에 흡수된다고 봤습니다. 근거 표는 README의 "대조 광역시 휴무일 이력"에 있습니다.
  • 인천은 2023년 4~12월에 대한 직접 근거가 "변경 보도 없음"뿐입니다.

🤖 Generated with Claude Code

jgkim1008 and others added 3 commits October 25, 2026 20:51
- README에 인천·대전·울산·부산·광주의 분석 기간(2019Q1~2023Q4) 휴무일 이력과 근거 기록
- 울산은 남·북·중구가 둘째 수요일 + 넷째 일요일로 기간 내내 같아 대조군에 유지
- 대조군 정의를 "의무휴업일을 바꾸지 않은 광역시"로 수정
- 울산을 뺀 추정(drop_unit)을 반박 검정으로 추가 (실행기 미지원 → 수동 실행)
- 판매액지수 원자료를 열기 전 커밋

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- data_sources: DT_1K41018(시도별 소매판매액지수, 분기), 항목 T2, 업종 A2·A4·A5, 지역 코드 기재
- 주 지표를 '슈퍼마켓· 잡화점 및 편의점'(A4)으로 수정. 시도 단위에서 편의점이 분리되지 않음
- 편의점 포함으로 인한 희석을 한계로 기록
- 판매액지수 값을 받기 전 커밋 (메타데이터만 조회)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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