Skip to content

Conversation

@YuujinHwang
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • dotenv를 통한 API 제공처 선택
  • LLM_PROVIDER 및 EMBEDDING_PROVIDER로 구분하여 Credential 설정
  • get_llm 및 get_embeddings 로 객체 생성 시 Provider에 맞는 wrapper 반환

💬 To Reviewers (선택)

  • langchain에서 기본으로 사용하는 환경변수와 일부 이름이 다릅니다.
  • embeddings와 llm이 동일 Provider 를 공유할 때, 별도로 세팅하는 부분에 불편함은 없을지.

PR Checklist

  • huggingface 및 gemini는 보유한 api키가 없어서 테스트해보지 못했습니다.
  • poetry로 프로젝트 의존성을 관리할 수 있도록 pyproject.toml을 업데이트하였습니다.

reference) How to Code Review

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

@YuujinHwang YuujinHwang changed the title add multi-llm provider Feature/41 add multi-llm provider Apr 21, 2025
Copy link
Contributor

@nonegom nonegom left a comment

Choose a reason for hiding this comment

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

👍다양한 모델을 사용할 수 있는 것 좋은 것 같습니다!!
💊추후 Class 형태로 관리가 되면 관리적인 측면에서 더 좋을 것 같습니다.

Copy link
Contributor

@seyoung4503 seyoung4503 left a comment

Choose a reason for hiding this comment

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

👍 다양한 llm 옵션을 제공하는게 좋은 것 같아요

@bongkyunSON
Copy link

❓: huggingface model들 중에는 가끔 형식을 다르게 불러오는 경우도 있던데 이부분은 크게 상관없을까요?

@anjaaaaeeeellll
Copy link
Contributor

💊 다양한 모델 테스트를 위해 LLM과 Embedding model 선택을 Streamlit에서 할 수 있게 만들어도 좋을 것 같습니다.

@ehddnr301 ehddnr301 merged commit 2a1ba84 into master Apr 24, 2025
2 checks passed
@ehddnr301 ehddnr301 deleted the feature/16-add-llm-provider branch September 7, 2025 07:08
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.

7 participants