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

Skip to content

[N-3] 13-LangChain-Expression-Language / 12-RunnableRetry #310

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 19, 2025
Merged

[N-3] 13-LangChain-Expression-Language / 12-RunnableRetry #310

merged 1 commit into from
Jan 19, 2025

Conversation

PangPangGod
Copy link
Contributor

  • RunnableRetry Bind with Chain 파트에 with_structured_output() 메소드와 RunnableRetry bind해서 사용하는 방법 및 이유에 대해 추가적으로 작성하였습니다.

  • 보시고 많은 피드백 및 approve도 부탁드립니다. 감사합니다.

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

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

@PangPangGod PangPangGod self-assigned this Jan 12, 2025
stsr1284
stsr1284 approved these changes Jan 13, 2025
View reviewed changes
Copy link
Contributor

@stsr1284 stsr1284 left a comment

Choose a reason for hiding this comment

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

🖥️ 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: 좋습니다! 고생많으셨습니다~

@seofield
Copy link
Contributor

seofield commented Jan 16, 2025 •
edited
Loading

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

저만 그런지 몰라도 Table of Contents 아래의 링크들이 동작 안합니다
그리고 OPENAI_API_KEY 가 없으면 중간부터 튜토리얼이 에러가 나는데요, 저 값이 있어야 진행 가능하다고 안내를 넣는건 어떤가합니다(굳이 안넣어도 됩니다~)
고생하셨습니다~

@PangPangGod
Copy link
Contributor Author

PangPangGod commented Jan 18, 2025 •
edited
Loading

@seofield

  • ToC 오류 없이 잘 동작하는거 확인했습니다!
  • 또 OPENAI_API_KEY 경우에는 with_chain 부분에서 실제 LLM Chain을 통한 예제를 구현했습니다.
  • gpt-4o-mini model 사용했기 때문에 필요하다고 생각해서 사용했습니다!

@teddylee777 teddylee777 merged commit 1beff1e into LangChain-OpenTutorial:main Jan 19, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seofield seofield seofield approved these changes

@stsr1284 stsr1284 stsr1284 approved these changes

Assignees

@PangPangGod PangPangGod

Labels
docs tutorial new tutorial suggestions
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@PangPangGod @seofield @stsr1284 @teddylee777 @sohyunwriter