-
Notifications
You must be signed in to change notification settings - Fork 282
12-RAG / 01-RAG-Basic-PDF revise #458
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.
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
12-RAG / 01-RAG-Basic-PDF revise #458
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
의견 반영한 것 확인했습니다~ 수고하셨습니다:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안녕하세요, 이번 검수 배정 받은 이준의입니다.
몇가지 수정 제안 사항이 있어서 정리해드립니다.
- Overview 섹션의 RAG Execution (RunTime) -> RAG Execution (Runtime) : Runtime을 있는 단어 그대로 수정 제안드립니다.
- Overview 섹션의 RAG Execution (RunTime) / Step 5 내용 중 '...are categorized as dense or sparse': 고유명사로 쓰인 케이스가 아니라 문장 내에서는 소문자로 표기하는 걸 제안드립니다.
- Reference 표기를 'LangChain How-to guides : Q&A with RAG'로 수정 제안드립니다.
- Environment-setup 섹션명 -> Environment Setup 으로 수정 제안드립니다.
- RAG Basic Pipeline 섹션 중 RAG의 약자 설명 -> Retrieval Augmented Generation 으로 수정 제안드립니다.
- 'Send a query to the retriever and check the resulting chunks.'라는 문구가 노트북 상 마크다운에도 있고 코드의 comment로도 있어서 comment 삭제 제안드립니다.
- 코드 상 '# Step 7: Create Language Model (LLM)'을 '# Step 7: Setup LLM'으로 수정 제안드립니다.
- Complete Code 섹션명의 Code의 첫 글자 대문자화 수정 제안드립니다.
- Complete Code 섹션 중 상기 언급된 수정 제안 사항 적용 제안드립니다.
수정 제안 커밋을 올렸으니, 확인 부탁드립니다.
Proofread 커밋 올린 것에 문제 없으면, 댓글 달아주시면 감사하겠습니다.
감사합니다.
@ee-juni |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제안 검토 완료하였습니다! 감사합니다.
@BokyungisaGod 제가 commit 새로 올린 거 땜에 보경님께서 approve한 게 취소가 되었나 보네요 ^^; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다:)
5a35472
into
LangChain-OpenTutorial:main
{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: 해당 이슈 수정하였습니다 (Proofread: <12-RAG / 01-RAG-Basic-PDF> #431)
❌ 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