Repository navigation
plan(jgkim1008-daegu-mart-holiday): 대조군 정의·주 지표 수정 (데이터 수집 전) - #45
Merged
jsshin2022 merged 3 commits intoOct 25, 2026
Merged
Conversation
- 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>
This was referenced Oct 25, 2026
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
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.
무엇을 했나요?
#34 계획에서 두 가지를 고쳤습니다. 둘 다 판매액지수 값을 받기 전에 커밋했습니다. 관련 이슈: #30
2c7571e): "일요일 의무휴업을 유지한 광역시"를 "분석 기간 중 의무휴업일을 바꾸지 않은 광역시"로 바꿨습니다.drop_unit)으로 추가했습니다. 공통 실행기가drop_unit을 아직 실행하지 않아 수동으로 돌립니다.d822589): KOSIS 메타데이터만 조회해 통계표를DT_1K41018(시도별 소매판매액지수, 분기)로 확정했습니다.discussion.md > 계획에서 바뀐 것에 위 내용을 기록했습니다 (3bfad00).케이스
cases/jgkim1008-daegu-mart-holiday체크리스트
plan.yaml을 결과를 보기 전에 작성·커밋했습니다 (이후 변경 시 이유를 커밋 메시지와discussion.md > 계획에서 바뀐 것에 기록)discussion.md > 결과를 한 문장으로를 채웠습니다 — 결과 PR 아님plan.yaml > data_sources에 명시했습니다 (KOGL-1. 통계표 화면에서 최종 확인 예정).env·개인정보가 담긴 원자료·재배포 불가 데이터를 포함하지 않았습니다fetch.py→make flow실행으로 재현 가능합니다 —fetch.py는 다음 PRmake check(ruff + pytest)가 통과합니다core/를 (합의 없이) 수정하지 않았습니다리뷰어에게
fetch.py로 데이터를 받았습니다. 수집할 때는 표 크기와 부산의 첫 3행만 출력해 형식을 확인했고, 대구와 비교하는 계산(make flow)은 하지 않았습니다.fetch.py는 이 PR이 합쳐진 뒤 별도 PR로 올립니다.🤖 Generated with Claude Code