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

Skip to content

Conversation

pupba
Copy link
Contributor

@pupba pupba commented May 7, 2025

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

Update

  • move legacy notebook and code
  • new light weight tutorial

- move legacy notebook and code
- new light weight tutorial
Copy link
Contributor

@sohyunwriter sohyunwriter left a comment

Choose a reason for hiding this comment

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

원저자명은 그대로 둬야할 것 같아 수정했습니다! 그리고 multimodal 부분도 우선 그대로 뒀어요!

@sohyunwriter sohyunwriter merged commit 6d31ceb into LangChain-OpenTutorial:main May 8, 2025
1 check failed
@pupba
Copy link
Contributor Author

pupba commented May 9, 2025

음...노트북 파일 내용, 코드들도 다 새로 로직짜서 작성된 내용들인데요.... 차라리 공동 저자가 맞지 않을까요...? (Qdrant도요..!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants