-
Notifications
You must be signed in to change notification settings - Fork 282
[N-3] 17-LangGraph / 01-Core-Features / 16-LangGraph-Errors #392
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
d7d5ba7
to
c747637
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그리고 전체적으로 backtick같은거 체크 한번만 더 해주시면 감사하겠습니다!
아마 검수팀에서 한번 더 볼 것 같긴 한데 전체적으로 빠져 있네요.
"cell_type": "markdown", | ||
"metadata": {}, | ||
"source": [ | ||
"# Erros\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분과 file name에 오타가 있네요!
# Erros
-> # Errors
16-LangGraph-Erros.ipynb
-> 16-LangGraph-Errors.ipynb
"\n", | ||
"### References\n", | ||
"\n", | ||
"- [LangGraph API Reference](https://langchain-ai.github.io/langgraph/reference/errors/)\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분도 LangGraph API Reference보다는 설명을 조금 더 해주면 좋을것 같아요
LangGraph API Reference
-> LangGraph API Reference: Errors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
이 튜토리얼에는 하기 키가 필요없어 보입니다.
"OPENAI_API_KEY": "",
"LANGCHAIN_API_KEY": "",
"LANGCHAIN_TRACING_V2": "true",
"LANGCHAIN_ENDPOINT": "https://api.smith.langchain.com",
"LANGCHAIN_PROJECT": "", -
마지막 빈셀들 삭제해주셔야할거 같아요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🖥️ OS: Windows
✅ 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: 피드백 하신 부분 확인했습니다. 고생하셨습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다
b5e2254
into
LangChain-OpenTutorial:main
{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: 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