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

Skip to content

Conversation

LEE1026icarus
Copy link
Contributor

{PR Message - Write freely, remove this bracket if unnecessary}

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.

  • (Optional) Related Issue: 해당 이슈 수정하였습니다 (Proofread: <12-RAG / 01-RAG-Basic-PDF> #431)

  • ❌ 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.

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, 한국어 기술 가능}     

If no one reviews your PR within a few days, please @-mention one of teddylee777, musangk, BAEM1N

BokyungisaGod
BokyungisaGod previously approved these changes Jan 20, 2025
Copy link
Contributor

@BokyungisaGod BokyungisaGod 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

@ee-juni ee-juni left a comment

Choose a reason for hiding this comment

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

안녕하세요, 이번 검수 배정 받은 이준의입니다.
몇가지 수정 제안 사항이 있어서 정리해드립니다.

  1. Overview 섹션의 RAG Execution (RunTime) -> RAG Execution (Runtime) : Runtime을 있는 단어 그대로 수정 제안드립니다.
  2. Overview 섹션의 RAG Execution (RunTime) / Step 5 내용 중 '...are categorized as dense or sparse': 고유명사로 쓰인 케이스가 아니라 문장 내에서는 소문자로 표기하는 걸 제안드립니다.
  3. Reference 표기를 'LangChain How-to guides : Q&A with RAG'로 수정 제안드립니다.
  4. Environment-setup 섹션명 -> Environment Setup 으로 수정 제안드립니다.
  5. RAG Basic Pipeline 섹션 중 RAG의 약자 설명 -> Retrieval Augmented Generation 으로 수정 제안드립니다.
  6. 'Send a query to the retriever and check the resulting chunks.'라는 문구가 노트북 상 마크다운에도 있고 코드의 comment로도 있어서 comment 삭제 제안드립니다.
  7. 코드 상 '# Step 7: Create Language Model (LLM)'을 '# Step 7: Setup LLM'으로 수정 제안드립니다.
  8. Complete Code 섹션명의 Code의 첫 글자 대문자화 수정 제안드립니다.
  9. Complete Code 섹션 중 상기 언급된 수정 제안 사항 적용 제안드립니다.

수정 제안 커밋을 올렸으니, 확인 부탁드립니다.
Proofread 커밋 올린 것에 문제 없으면, 댓글 달아주시면 감사하겠습니다.

감사합니다.

@LEE1026icarus
Copy link
Contributor Author

@ee-juni
안녕하세요 선생님, 선생님이 수정 제안 해주신 것을 검토하였습니다.
많이 부족한 코드 리뷰에 언제나 감사드립니다!

Copy link
Contributor Author

@LEE1026icarus LEE1026icarus left a comment

Choose a reason for hiding this comment

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

제안 검토 완료하였습니다! 감사합니다.

@LEE1026icarus LEE1026icarus requested a review from ee-juni January 23, 2025 01:45
@ee-juni
Copy link
Contributor

ee-juni commented Jan 23, 2025

@BokyungisaGod 제가 commit 새로 올린 거 땜에 보경님께서 approve한 게 취소가 되었나 보네요 ^^;
다시 한번 더 approve 부탁드립니다!

Copy link
Contributor

@BokyungisaGod BokyungisaGod left a comment

Choose a reason for hiding this comment

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

수고하셨습니다:)

@teddylee777 teddylee777 merged commit 5a35472 into LangChain-OpenTutorial:main Jan 23, 2025
3 of 4 checks passed
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.

4 participants