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

Skip to content

Conversation

BokyungisaGod
Copy link
Contributor

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

choincnp
choincnp previously approved these changes Jan 16, 2025
Copy link
Contributor

@choincnp choincnp left a comment

Choose a reason for hiding this comment

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

🖥️ OS: Win
✅ 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:
    안녕하세요 @BokyungisaGod 님, 튜토리얼 제작하시느라 고생 많으셨습니다!
    제 기준에서는 Code Execution, Templates, Image 등 조건 다 만족했습니다.

Copy link
Contributor

@forwardyoung forwardyoung left a comment

Choose a reason for hiding this comment

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

🖥️ OS: Win
✅ 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:
    @BokyungisaGod 수고하셨습니다:) 이미지 파일명만 소문자로 바꿔주심 될 것 같습니다.

choincnp
choincnp previously approved these changes Jan 16, 2025
Copy link
Contributor

@choincnp choincnp left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다 :)

@BokyungisaGod
Copy link
Contributor Author

@forwardyoung 수정했습니다~!

@forwardyoung
Copy link
Contributor

forwardyoung commented Jan 17, 2025

안녕하세요! 혹시 번거로우시겠지만 이미지를 delete했다가 다시 올려주실 수 있나요? 아직 대문자로 남아있고, ipynb 파일에는 이미지가 안 보여서요!
image

@BokyungisaGod
Copy link
Contributor Author

@forwardyoung 코멘트를 좀 전에 확인해서ㅜㅜ 이제 막 커밋 했네요. 변경 완료 했습니다!

@BokyungisaGod
Copy link
Contributor Author

커밋이 꼬여서 새로 PR 올립니다. #427 에 새로 PR 오픈했습니다!

@sohyunwriter sohyunwriter added the docs tutorial label Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs tutorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants