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

Skip to content

Conversation

geminii01
Copy link
Member

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.

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

@geminii01 geminii01 added docs tutorial proofreading 번역/검수팀 제안사항 반영 labels Jan 22, 2025
@geminii01 geminii01 self-assigned this Jan 22, 2025
Copy link
Contributor

@Hye-yoonJeong Hye-yoonJeong 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
✅ 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:
  • ToC 중 오탈자 Environement Setup -> Environment Setup
  • 질문 :
  1. Setup the Graph 중 tool 정의하고 bind 하는 부분에서 def search_keyword 와 tools = [search_keyword] 사이에 news_tool = GoogleNews() 이거는 필요 없는 줄 아닌지 확인 부탁드립니다.
  2. Graph 정의 중 Add the Node -> Add Nodes, Add the Edge -> Add Edges

@geminii01
Copy link
Member Author

geminii01 commented Jan 23, 2025

  • Comments:
  • ToC 중 오탈자 Environement Setup -> Environment Setup
  • 질문 :
  1. Setup the Graph 중 tool 정의하고 bind 하는 부분에서 def search_keyword 와 tools = [search_keyword] 사이에 news_tool = GoogleNews() 이거는 필요 없는 줄 아닌지 확인 부탁드립니다.
  2. Graph 정의 중 Add the Node -> Add Nodes, Add the Edge -> Add Edges

@Hye-yoonJeong
안녕하세요!
오타, 불필요한 코드, 주석 3가지 수정사항 반영하였습니다.
시간되실 때 재검토 부탁드립니다.
감사합니다!

Hye-yoonJeong
Hye-yoonJeong previously approved these changes Jan 23, 2025
Copy link
Contributor

@Hye-yoonJeong Hye-yoonJeong left a comment

Choose a reason for hiding this comment

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

수정사항 반영과 코드 정상작동함을 확인하였습니다! 고생많으셨습니다! :)

@ThePurpleCollar
Copy link
Contributor

오류 없이 깔끔하게 작성된 코드와 설명이 인정적이네요. 고생하셨습니다.

Copy link
Contributor

@chaeyoonyunakim chaeyoonyunakim left a comment

Choose a reason for hiding this comment

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

번역/검수 확인했습니다. 잘 쓰여진 문장들이라 오히려 리뷰 시간이 더 오래 걸린 것 같습니다. 소제목 중 Setup the Graph가 전후 제목 형식과 달라 통일하고자 몇 가지 대안을 제안드립니다: Setting up the Graph 혹은 Graph Setup, Graph Configurations, Graph initialization 등이 가능합니다. 제가 commit하여 approval과정을 다시 거칠 만큼 사안은 아니라 comment로 대신합니다. 감사합니다.

@geminii01
Copy link
Member Author

번역/검수 확인했습니다. 잘 쓰여진 문장들이라 오히려 리뷰 시간이 더 오래 걸린 것 같습니다. 소제목 중 Setup the Graph가 전후 제목 형식과 달라 통일하고자 몇 가지 대안을 제안드립니다: Setting up the Graph 혹은 Graph Setup, Graph Configurations, Graph initialization 등이 가능합니다. 제가 commit하여 approval과정을 다시 거칠 만큼 사안은 아니라 comment로 대신합니다. 감사합니다.

@chaeyoonyunakim @Hye-yoonJeong @ThePurpleCollar
안녕하세요!
제목 형식을 Graph Setup 으로 변경했습니다. 목차 예쁘게 잡아주셔서 감사합니다!
시간되실 때 재검토 부탁드립니다.
감사합니다!

@teddylee777 teddylee777 merged commit aafb7d5 into LangChain-OpenTutorial:main Jan 25, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs tutorial proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants