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

Skip to content

Conversation

ulysyszh
Copy link
Contributor

@ulysyszh ulysyszh commented Jan 9, 2025

{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)

🖥️ 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

@ulysyszh ulysyszh requested review from gyjong and ee-juni January 9, 2025 14:39
@ee-juni ee-juni requested review from BokyungisaGod and removed request for gyjong January 10, 2025 00:10
@ee-juni ee-juni added the proofreading 번역/검수팀 제안사항 반영 label Jan 10, 2025
Copy link
Contributor

@ee-juni ee-juni left a comment

Choose a reason for hiding this comment

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

안녕하세요~ 😊
몇가지 추가적으로 검토 요청드리고 싶은 사항이 있어 다시 한번 확인 부탁드립니다.

1. 파일명 변경

기존에 04-로 시작하는 파일명이 04_로 변경된 것으로 보입니다.
기존 명칭으로 원상복구 부탁드립니다!

2. Peer review 부분 수정

노트북 맨 위에 나와 있는 부분에서 design 항목이 별도로 포함되지 않았기에 해당 줄을 삭제하고,
아래와 같은 형식으로 peer review 부분을 수정 부탁드리겠습니다:
- Peer Review: [rlatjcj](https://github.com/rlatjcj), [gyjong](https://github.com/gyjong)

3. Pull Request 명칭

이 부분은 현재 issue와 직접적인 관련은 없지만, pull request 제목에 일정한 규칙이 있는 것으로 알고 있습니다.
제가 개발팀 소속이 아니어서 정확한 규칙은 잘 모르지만, 가능하시다면 한번 확인 부탁드리겠습니다!
혹시 추가로 필요한 내용이 있으시면 말씀 주세요. 감사합니다! 🙏

@ulysyszh
Copy link
Contributor Author

ulysyszh commented Jan 10, 2025

감사합니다~ 준의님, BokyungisaGod님

  1. 파일명 변경했습니다.
  2. Peer review 부분 추가했습니다.
  3. Pull Request 명칭 정하는 규칙을 모르겠습니다...

@teddylee777 teddylee777 changed the title Reflecting suggested improvements 05-Memory / 04-ConversationEntityMemory Jan 12, 2025
@teddylee777 teddylee777 merged commit bf28fa9 into LangChain-OpenTutorial:main Jan 13, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants