Skip to content

[PR머신 후속] PR #6 P2 개선 — feat: 예측 결과 CSV 요약 리포트 유틸리티 추가 #7

Description

@claude

🤖 PR #6 (@vapsnamheo-dev) 자동 머지 시 발견된 P2 개선 항목입니다.
머지는 됐지만, 아래 항목을 추후 처리해 주세요. (P3는 선택 개선으로 함께 참고)

체크리스트 (P2)

  • [P2] summarize_csv에서 파일 읽기 예외를 조용히 무시하고 정상 응답처럼 반환 Homework/ML_FactoryAutomation/src/report_util.py:8
    • open(path) 실패(FileNotFoundError, PermissionError 등)를 except Exception: pass로 삼켜 rows가 빈 리스트로 남고, 이후 로직이 이를 "빈 CSV"와 동일하게 처리해 {"count": 0, "grade": "N/A"}을 반환합니다. 호출자는 파일이 없어서 실패했는지 실제로 데이터가 없는지 구분할 수 없어 경로/권한 문제가 조용히 숨겨집니다.
  • [P2] save_summary 기본 출력 경로가 Windows 전용 절대경로로 하드코딩됨 Homework/ML_FactoryAutomation/src/report_util.py:40
    • out_dir="C:/temp/pdm_reports"는 Windows 드라이브 표기입니다. Linux/Mac에서 기본값으로 호출하면 절대경로로 인식되지 않고 현재 작업 디렉터리 기준 상대경로가 생성되어, 크래시 없이 예상치 못한 위치에 리포트가 쌓입니다.

참고 (P3)

  • [P3] float 파싱 실패를 조용히 스킵 Homework/ML_FactoryAutomation/src/report_util.py:20-24
    • 숫자가 아닌 값은 건너뛰는 것은 의도된 동작으로 보이나, 스킵된 행 수를 로깅하지 않아 데이터 품질 저하를 추적하기 어렵습니다.
  • [P3] 등급 임계값(0.75, 0.35)이 매직 넘버로 하드코딩됨 Homework/ML_FactoryAutomation/src/report_util.py:30-33
    • 임계값 변경 시 코드를 직접 찾아 고쳐야 합니다. 이름 있는 상수로 추출하면 의도가 명확해집니다.
  • [P3] save_summary의 파일 쓰기에 예외 처리 없음 Homework/ML_FactoryAutomation/src/report_util.py:45
    • 쓰기 권한이 없거나 디스크 공간이 부족한 경우 처리되지 않은 예외가 그대로 전파됩니다.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    ai:p2-followupPR머신 P2 후속 - 이슈 자동 등록 트리거

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions