Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 튜토리얼대로 실행시 발생했던 에러로그를 수정 및 문서로 정리

💬 To Reviewers (선택)

  • README.md 의 "빠른 설치" 와 "소스에서 설치" 시 파이썬 버전을 통일했습니다.
  • 튜토리얼대로 실행시 아래 에러가 psycopg[binary] 관련 에러가 발생해 pyproject.toml 파일을 수정했습니다.
ImportError: no pq wrapper available.
Attempts made:
- couldn't import psycopg 'c' implementation: No module named 'psycopg_c'
- couldn't import psycopg 'binary' implementation: No module named 'psycopg_binary'
- couldn't import psycopg 'python' implementation: libpq library not found
  • 튜토리얼대로 실행시 .env 파일을 읽지않고 로컬의 환경변수를 읽어서, .env 내용이 환경변수에 없는 경우 실패해 dotenv 를 사용해 create_faiss.py가 실행하도록 수정했습니다.
from dotenv import load_dotenv
load_dotenv()
  • 튜토리얼대로 실행시, DB_TYPE이 없을 때 실행되지 않는 문제가 있어서 문서에 DB_TYPE을 기본값으로 추가했습니다.
ValueError: DB type must be provided or set in environment as DB_TYPE.
  • 튜토리얼의 최상단에 환경구성부터 있으면 좋을 것 같아 준비단계를 추가했습니다.

PR Checklist

  • 제 로컬에서 테스트시에는 문제 없었습니다.

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

"mysql-connector-python>=9.3.0,<10.0.0",
"duckdb>=1.2.2,<2.0.0",
"psycopg2-binary>=2.9.10,<3.0.0",
"psycopg[binary]>=3.2,<4.0",
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 pyproject.toml에 없는데 왜 제 환경에선 되는지 이상해서 history를 보니까 uv pip install "psycopg[binary]" 이렇게 설치한 기록이 있네요ㅋㅋㅋ큐ㅠ 감사합니다. uv add 로 설치해야 반영이 되는군요

# PGVECTOR_COLLECTION=table_info_db

# DB 타입
DB_TYPE=clickhouse
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 DB_TYPE 관련 에러는 Optional 하게 수정이 되어야겠네요ㅎㅎ 발견 감사합니다.

@ParkGyeongTae ParkGyeongTae merged commit 5812f6a into master Sep 14, 2025
2 checks passed
@ParkGyeongTae ParkGyeongTae self-assigned this Sep 15, 2025
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.

3 participants