-
Notifications
You must be signed in to change notification settings - Fork 282
[N-1] 06-DocumentLoader / 03-WebBaseLoader.ipynb #436
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
[N-1] 06-DocumentLoader / 03-WebBaseLoader.ipynb #436
Conversation
꼼꼼히 검수해주셔서 감사합니다. 아래 사항 반영하여 커밋합니다. - pip install 부분에서 !를 %로 변경하였습니다. - 가이드 주신 typo 부분들 반영하였습니다. - 제목에 쓰인 백틱은 제거하였습니다.
안녕하세요 @HarryKane11 님:) 우선 수정해주신 부분은 모두 확인하였습니다. 추가적으로 몇군데만 더 수정해주시면 감사하겠습니다. Simple Web Content Loading with Unlike Below is a basic example of loading web content using +) 현재 저희 팀원의 검수 내용과 제가 따로 검수해본 결과 마크다운 문법(특히 backtick)을 거의 고려하지 않으신 것 같습니다.. 저희 검수팀은 개발을 꾸준히 하시던 분들로 구성되어있지 않은지라 개발을 하시는 분들과 비교했을 때, 변수, 함수, 모듈명이 헷갈리는 부분 많습니다. 특히, 모듈명과 라이브러리 명에서요. 물론 번역/검수팀에서 최대한 꼼꼼히 검수하고, 수정해야 할 부분들을 찾아 feedback을 드리는 것이 가장 이상적인 것은 알고 있습니다. 하지만, 이제 3주차가 지나가는 시점에 markdown 문법같은건 특히 조금 더 신경써서 내용을 작성해주셨으면 합니다! (함수/변수/모듈명 관련한 내용은 개발하신 분들이 저희보다 훨씬 더 잘 아실 것 같다는 생각이 들어요ㅜㅜ.) 부탁드립니다:) |
안녕하세요 @BokyungisaGod 보경님! 우선 검수팀에서 많은 문서를 검수하시느라 고생하시는거 잘 알고, 제 문서에 틀린 부분이 많아 번거로웠을 심정 이해하고 죄송하다는 말씀 드립니다. 제가 backtick 관련 부분에서 오류가 많아보이는건, 디스코드 상에서 히스토리 따라가면서 확인하다가, 제목에서는 backtick을 안해도 되지 않나 하는 의견(가이드-PPT 마스터 가이드 1월 17일자에 백틱 관련 규칙 정리하신다는 글)을 보고 그렇게 고쳤었습니다. 제목에도 변수명이면 backtick을 쓰는 것은 이미 컨센서스 된 부분이라면 충분히 Follow-up하지 못한 것 같네요... 주의하겠습니다. 아래 번호별로 의견 드립니다.
|
- 마크다운 문법 적용 안된 부분이나 잘못 처리된 부분 Fix
- 튜토리얼 링크가 삭제되어 있어서 다시 추가했습니다.
넵넵 의견 반영 감사합니다!! (참고: 소제목의 경우도 앞으로 backtick처리 할 것 같아요!) |
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.
수고하셨습니다:)
5856175
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 Proofread: < 06-DocumentLoader / 03-WebBaseLoader.ipynb > #418 )
❌ 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