Skip to content

Conversation

@ash-hun
Copy link

@ash-hun ash-hun commented Jan 18, 2026

PR 타입

  • 새로운 튜토리얼 추가
  • 기존 튜토리얼 수정 (버그 수정 / 내용 개선 / 리팩토링)
  • 문서 수정
  • 기타 (chore, 의존성 업데이트 등)

변경 사항

  • LangGraph 그래프 생성 튜토리얼 내용 업데이트

수정된 파일

  • 02-Basic/01-LangGraph-Models.ipynb

체크리스트

기존 튜토리얼 수정인 경우

  • 금지 파일 미포함
  • 노트북 셀 에러 출력 없음
  • 파일명 형식 준수 ({NN}-LangGraph-{Topic}.ipynb)
  • 불필요한 출력/주석 없음
  • 오탈자 및 문장 오류 검토
  • AI 흔적 없음 (커밋 이력)
  • PR 미리보기 확인 완료

테스트 결과

테스트 환경:

  • OS: macOS
  • Python: 3.x

추가 설명

아래 내역 추가 의견주시면 반영하겠습니다.

  • Jupyter Kernel 생성시 psycopg2 부분 호환성 에러가 발생하여, 타 작업자분들이 어떻게 테스트하시는지 모르겠으나 psycopg-binary로 호환성 문제를 해결하여 pyproject.toml과 uv.lock을 업데이트 해놓았습니다.
  • 최하단에 OpenAI API 호환파트의 경우 LMStudio, vllm, ollama의 호환파트로 처음 받았을 때 각주처리되어있어 건드리지 않았습니다. Reviewer께서 확인하시고 의견주시면 논의하여 반영하면 좋을 것 같습니다.

@ash-hun ash-hun requested a review from LHANTAEK January 18, 2026 05:37
@ash-hun ash-hun self-assigned this Jan 18, 2026
Copy link
Contributor

@LHANTAEK LHANTAEK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

추가 커밋된 부분 검토해주시고 머지해주시면 될 것 같습니다.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[제안] 해당 Trace 링크는 기간 만료가 되어서 접근할 수 없네요. 첨부한 사진으로 대체하는게 어떨까 싶습니다.

Image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

./assets/ 에 추가해서 마크다운 변경 후 커밋 올려놓겠습니다.

"notebook>=7.4.7",
"pandas>=2.3.3",
"psycopg2>=2.9.11",
"psycopg2-binary>=2.9.11",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

재훈님이 말한대로 psycopq2는 libpq-dev나 C 컴파일러 등 시스템 의존성이 없는 환경에서는 설치 실패가 발생할 수 있을 것 같습니다. 튜토리얼 목적으로 제안해주신 psycopq2-binary 가 적합할 것 같습니다. 감사합니다.

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