Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

HarryKane11
Copy link
Contributor

[N-1] 06-Multimodal/ 10-GeminiMultimodalRAG

  • 필요 API키: Google, Upstage, OpenAI

Author Checklist

  • PR Title Format: I have confirmed that the PR title follows the correct format. (e.g., [N-2] 07-Text Splitter / 07-RecursiveCharacterTextSplitter)

  • Committed Files: I have ensured that no unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) are included. These files are not allowed.

  • ❌ Do not include unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) or other people's code. If included, close the PR and create a new PR.

@HarryKane11 HarryKane11 requested review from brian604 and obov January 8, 2025 14:28
@obov
Copy link
Contributor

obov commented Jan 9, 2025

Review Template (Intial PR)

🖥️ OS: Win/Mac/Linux   
✅ Checklist      
 - [ ] **Template**: Tutorials follows the required template. 
 - [ ] **Table of Contents(TOC) Links**: All Table of Contents links work. ((Yes/No)
 - [ ] **Image**: Image filenames follow guidelines.
 - [ ] **Imports*: All import statements use the latest versions. Ensure "langchain-teddynote" is not used. 
 - [ ] **Code Execution**: Code runs without errors.
 - Comments: {Write freely, 한국어 기술 가능}     

obov
obov previously approved these changes Jan 9, 2025
Copy link
Contributor

@obov obov left a comment

Choose a reason for hiding this comment

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

🖥️ OS: Mac
✅ Checklist

  • Template: Tutorials follows the required template.
  • Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
  • Image: Image filenames follow guidelines.
  • *Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
  • Code Execution: Code runs without errors.
  • Comments:
    아직 draft이니 포맷관련 내용과 기술적인 부분 구분해서 리뷰드립니다

포맷

  1. 주석 : 한글 -> 영어로 변경필요합니다.
  2. 추가 설치 라이브러리 : pip -> langchain_opentutorial 리스트로 추가
  3. set_env 리스트에 UPSTAGE_API_KEY 추가
  4. TOC 링크 확인 필요합니다.
  5. 문서 제목으로 gemini가 들어가있는데 주도적인 역할을 하는지 다시 한번 검토 부탁드립니다.

기술

  1. 도식화 이미지가 제공되면 전체적인 답변생성 구조를 이해하는데 도움이 될것 같습니다. 참고 링크
  2. 마지막부분의 예시 질문의 답변 예시를 바꾸거나 로직을 확인해주시면 감사드립니다. 제가 문서를 제대로 이해하지 못해서 그럴수도 있긴한데,
image 문서에서 display 된 image 예시의 일부입니다. 해당내용에는 많은 국가가 포함되어 있는데 여러번 질문 생성을 해봐도 `India, Saudi Arabia, and the UAE` 이렇게 네개 국가만 계속해서 출력됩니다. 제가 확인해본바로는 retrieval 되는 문서중 첫번째에 포함된 내용만 사용해서 답변중입니다.

살펴보시고 적용 필요하다고 생각하시는 부분에 대해 반영해주시면 감사드립니다.
정확히 설명하느라 내용이 길어진 것이고 다음주에는 좋은 코드예시가 될 것 같습니다.
고생하셨습니다. 감사합니다 ~

- 한글주석을 영어주석으로 번역
- 아키텍처 그래픽 및 Upstage Document Parse 이미지 추가
- 예제 코드 변경하여 올바른 출력 유도
- ToC 수정
- 필요 pip install 패키지 추가
obov
obov previously approved these changes Jan 16, 2025
Copy link
Contributor

@obov obov left a comment

Choose a reason for hiding this comment

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

[Review Checklist]

  • Review OS: Mac

  • Template Rule 준수 여부:

  • Table of Contents 링크 확인:

  • 이미지 파일명 검토: 이미지 파일명 확인 한번 부탁드립니다!

image
  • 최신 import 방식 사용 여부:

  • 코드 동작 확인:

  • 제출 파일 확인:

  • 기타 의견:
    다양한 api를 활용해서 RAG 구현해주신게 인상적이었습니다. LangGraph 구현 에시로 참고하기 좋은 코드인 듯 합니다!
    2주간 고생하셨습니다

가이드라인에 맞춰 이미지 파일명 수정하였습니다.
Copy link
Contributor

@brian604 brian604 left a comment

Choose a reason for hiding this comment

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

🖥️ OS: Win/Mac/Linux
✅ Checklist

  • [ x ] Template: Tutorials follows the required template.
  • [ x ] Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
  • [ x ] Image: Image filenames follow guidelines.
  • [ x ] *Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
  • [ x ] Code Execution: Code runs without errors.
  • Comments: {Write freely, 한국어 기술 가능}
    Code execution has not been checked as a whole, due to quota limit.
    However, it would be great if we use `` to call for function, API, or packages, etc; such as pymupdf4llm as pymupdf4llm.
    @HarryKane11 good work. You inspire me with how to handle modality data other than text.

Copy link
Contributor

@brian604 brian604 left a comment

Choose a reason for hiding this comment

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

I will approve, because those formatting changes may look trivial to content creator.

@teddylee777 teddylee777 merged commit dc1090b into LangChain-OpenTutorial:main Jan 19, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants