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

Skip to content

Conversation

jhboyo
Copy link
Contributor

@jhboyo jhboyo commented Jan 26, 2025

Issue ticket

  1. 수정해주신 링크 참고하여 대부분 수정하였습니다.

  2. Langchain docs를 참고하여 기존 Define the tool 에서 Creating tools로 변경해봤는데 검토 부탁 드립니다.

  3. Creating tools 섹션에서 함수 를 설명하는 부분은 가독성을 위해서 Args와 Return부분은 제거하여 보았습니다. 바로 밑줄에도 Args와 Returns가 설명되기 때문입니다.

위 2번, 3번을 제외하고 수정해주신 링크 참고하여 수정하였습니다.

링크에 수정 부분이 너무 많아서 죄송한 마음이 큽니다. 늦은시간까지 고생 해주셔서 감사 드리며,
추가 수정할 부분이 나오면 또 업데이트 부탁 드리겠습니다.

감사합니다.

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: If this PR is linked to an issue, I have referenced the issue number in the PR message. (e.g., Fixes Update 01-PromptTemplate.ipynb #123)

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

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

@jhboyo jhboyo added the proofreading 번역/검수팀 제안사항 반영 label Jan 26, 2025
@teddylee777 teddylee777 merged commit 23ef507 into LangChain-OpenTutorial:main Feb 1, 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
proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants