[E-4] 17-LangGraph / 03-Use-Cases / 06-Multi-Agent-Collaboration #547
+709
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
delete python file and modify query
jinucho
approved these changes
Jan 24, 2025
There was a problem hiding this comment.
🖥️ 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: 고생하셨습니다.
|
🖥️ OS: Win
|
Secludor
approved these changes
Jan 24, 2025
chaeyoonyunakim
approved these changes
Jan 25, 2025
There was a problem hiding this comment.
안녕하세요, 번역/검수 확인 완료하였습니다. 수정했을 수도 있는 두 가지 사항에 대해 변경 없이 승인하는 배경을 남겨둡니다.
-
(다른 튜토리얼 노트북들 포함) 대부분 소제목들이 [명사(절)]로 통일되어 있습니다. 보통은 검수과정에 영문법적 오류를 수정하시며 덤으로 같이 수정해달라 요청드리는 편입니다; "Why do we use multi-agent networks?" -> "Benefits of Multi-Agent Networks". 그런데 이번 노트북은 이것을 제외하면 수정할 부분이 전혀 없고, 환경 설정 이후 바로 튜토리얼 개시 부분이기 때문에 유지해도 되겠다는 판단이 들었습니다. 양식 통일을 하면서 몇몇 좋은 질문들과 개성이 사라지는게 아쉬웠는데 이번은 오히려 잘 되었다 생각합니다.
-
TavilySearch가
함수,변수,모듈명backtick 적용 대상이 아닌 것으로 보입니다. 설명 강조 적용을 하는게 맞을 수도 있겠지만 뒤따라오는TavilySearchResults의 축약으로 어쩌면 볼 수도 있겠다 판단하여 유지할 수 있을 듯 합니다.
좋은 원고 감사합니다.
35c8c57
into
LangChain-OpenTutorial:main
3 of 4 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
이번주에 진행된 langchain_teddynote 패키지 내재화 작업을 반영하여
multi_agent_collaboration.py 삭제 및 관련 패키지 import 구문 수정되었습니다.
추가로 마지막 셀에 agent invoke 쿼리가 매년 탄소배출량 관련 내용이었는데,
관련 값을 웹검색으로 찾기가 어려워서 그런지 가끔씩 recursion_limit에 걸려서 에러가 났습니다.
그래서 쿼리를 글로벌 1인당 GDP 내용으로 바꿨습니다.
그 이외에는 이전과 달라진 내용 없습니다!
새로 PR 열어서 번거로우시겠지만 마지막 리뷰 부탁드립니다~
감사합니다.
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)
If no one reviews your PR within a few days, please @-mention one of teddylee777, musangk, BAEM1N