[E-3] 12-RAG / 02-RAG-Basic-Webloader #255
Conversation
There was a problem hiding this comment.
🖥️ OS: Win
✅ Checklist
- Template: Tutorials follows the required template.
- Table of Contents(TOC) Links: All Table of Contents links work. (Yes)
- 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, 한국어 기술 가능}
Comments
Reference의 LangChain WebLoader Tutorial을 클릭하면, 오류가 발생합니다.
bold체나 backtic 다음에는 띄어쓰기가 있어야 합니다.
pip command는 할떄는 ! 대신 %로 대체해주세요.
Reference 밑에 --- 코드를 추가해 주세요.
고생하셨습니다.
There was a problem hiding this comment.
수고하셨습니다.
Review Template (Intial PR)
🖥️ OS: Win
✅ Checklist
- Template: Tutorials follows the required template.
- Table of Contents(TOC) Links: All Table of Contents links work. ((Yes)
- 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, 한국어 기술 가능}
몇 가지 의견을 드렸습니다.
12-RAG/02-RAG-Basic-WebLoader.ipynb
Outdated
| "id": "02147e1b", | ||
| "metadata": {}, | ||
| "source": [ | ||
| "If you practice in Naver-News URL, you can download and input the `teddynote/rag-prompt-korean` prompt from hub. In this case, the separate prompt writing process can be skipped.\n", |
There was a problem hiding this comment.
teddynote는 가급적 사용하지 않고 진행해 달라고 공지가 있었어요.
prompt = hub.pull("teddynote/rag-prompt-korean") 는 빼시는게 어떨까 합니다.
|
@sunworl, @Jae-hoya 리뷰 감사드립니다! 말씀 주신 내용 수정 사항 반영하고 이미지명 수정하였습니다! 재리뷰 부탁드립니다..!😅 @sunworl 튜토리얼 내용은 forbes 문서 내용을 기준으로 가져와서 실습하는 방식으로 구성하였는데, (cc. @teddylee777) |
고생하셨습니다! |
|
한국적인것과 'teddynote'가 들어가는 것에 대해 배제를 말씀하셨던지라...
나중에 추가작업 발생을 예방하는 차원에서 확인하면 좋을듯합니다. 나머지 수정사항들은 확인했습니다. 너무 수고하셨습니다. |
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)
- 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, 한국어 기술 가능}
- Steps 1 to 8까지의 텍스트와 이미지 내 텍스트의 내용을 통일하면 더 좋지 않을까 생각합니다.
WebBaseLoader로더 사용 시USER_AGENT설정이 되어 있지 않아 경고가 출력되는 것으로 보이는데 set_env 단계에서"USER_AGENT":"Mozilla/5.0 (compatible; langchain-opentutorial/1.0; +https://github.com/langChain-OpenTutorial/)"이런 식의 추가가 있으면 좋을 거 같습니다.USER_AGENT environment variable not set, consider setting it to identify your requests.
- 제가 보기에는
hub.pull("teddynote/rag-prompt-korean")부분은langchain-teddynote라이브러리의 사용을 제한하는 거라 관련이 없어 보이고 프롬프트를 받아오는 ID라서 사용해도 무방합니다만 콘텐츠가 글로벌 콘텐츠를 목표하니 지금처럼 안내 정도만 나가도 충분할 거 같습니다. - 이건 그냥 궁금증인데 파일명이
02-RAG-Basic-WebBaseLoader가 아니라02-RAG-Basic-Weblodaer인 이유가 있을까요??
a2de521
|
@BAEM1N 말씀주신 경고문 관련 문구 추가하였고, 이미지와 맞게 설명글 수정하였습니다! |
d5616d7
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: 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