[E-1] 10-Retriever / 05-ParentDocumentRetriever.ipynb #260
Conversation
[Title] Existing content development team 1 [Version] initial [Language] ENG [Packages] langchain-openai, langchain-chroma Tutorial for Memory Using Parent Document Retriever in LangChain * Add tutorial and data(txt file)
There was a problem hiding this comment.
[Review Checklist]
-
[* ] Review OS:
- Windows / Mac / Linux (해당 OS를 기재해주세요) win
-
[* ] Template Rule 준수 여부:
- Template Rule 가이드를 준수하였는가? (Check 표기)
-
[ *] Table of Contents 링크 확인:
- Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
-
[ *] 이미지 파일명 검토:
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
-
[* ] 최신 import 방식 사용 여부:
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
-
[* ] 코드 동작 확인:
- 모든 코드가 오류 없이 동작하는가? (Check 표기)
- Warning 발생 시 코멘트에 적어주세요.
-
[ *] 제출 파일 확인:
- 꼭 필요한 파일들만 제출되었는가?
- 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
- 꼭 필요한 파일들만 제출되었는가?
-
기타 의견:
추가적인 의견이나 수정 사항을 기재해주세요.
loader = TextLoader("./data/appendix-keywords.txt", encoding="utf-8")
아래 주석으로 위 한줄 넣어두셔도 되겠습니다.
window 컴퓨터는 이렇게 되어야 하지만 일단 승인합니다.
|
@3DKIDS 리뷰 감사합니다! 해당 부분 추가하여 다시 커밋하였습니다~ 감사합니다 |
There was a problem hiding this comment.
🖥️ OS: Mac
✅ Checklist
- Template: Tutorials follows the required template.
- Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
- *Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
- Code Execution: Code runs without errors.
Comments:
코드 전부 정상 작동합니다. 👍
사소한 backtick 추가 등은 suggestion으로 남겨두었고,
TOC 링크는 전부 잘 동작하지만, Environments 항목이 하나 빠진 것 같아 suggestion으로 추가했습니다.
확인 후 커밋해주시면 바로 approve하겠습니다!
There was a problem hiding this comment.
🖥️ OS: Mac
✅ Checklist
- Template: Tutorials follows the required template.
- Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
- *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.
[Review Checklist]
-
Review OS: Windows
-
Template Rule 준수 여부: 이상없음
-
Table of Contents 링크 확인: 이상없음
-
이미지 파일명 검토: 해당없음
-
최신 import 방식 사용 여부:
-
코드 동작 확인: 정상작동합니다.
- comment 로 안내해주신대로, 윈도우에서는 encoding='utf-8' 옵션 추가하면 실행 잘 됩니다.
-
제출 파일 확인: 파일 위치 확인 완료
-
기타 의견:
리뷰어팀 전창원입니다.
코드는 오류없이 작동 잘 되었습니다.
파일 로딩 시 코멘트로 안내된 대로 추가 옵션을 넣으면 윈도우에서도 이상 없었습니다.
다만, 사용 패키지가 실제로 코드에 사용되는 것인지 한번 확인 부탁드립니다.
작성하시느라 고생 많으셨습니다! 감사합니다.
There was a problem hiding this comment.
[Review Checklist]
-
Review OS: Windows
-
Template Rule 준수 여부: 이상없음
-
Table of Contents 링크 확인: 이상없음
-
이미지 파일명 검토: 해당없음
-
최신 import 방식 사용 여부:
-
코드 동작 확인: 정상작동합니다.
- comment 로 안내해주신대로, 윈도우에서는 encoding='utf-8' 옵션 추가하면 실행 잘 됩니다.
-
제출 파일 확인: 파일 위치 확인 완료
-
기타 의견:
리뷰어팀 전창원입니다.
이전 의견 드린 부분 해결되어 approve 확인합니다.
고생 많으셨습니다. 감사합니다!
38ff773
into
LangChain-OpenTutorial:main
[Title] Existing content development team 1
[Version] initial
[Language] ENG
[Packages] langchain-openai, langchain-chroma
Tutorial for Memory Using Parent Document Retriever in LangChain
{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