[N-1] 06-DocumentLoader / 07-PowerPoint Loader, 08-WebBaseLoader #105
Conversation
[Title] PowerPoint-Loader [Version] fix [Language] ENG [Packages] langchain-community, unstructured, markitdown - poetry, pyproject 제외 후 다시 커밋 [Title] Webbase-Loader [Version] initial [Language] ENG [Packages]langchain_community, markitdown - poetry, pyproject 제외 후 다시 커밋
There was a problem hiding this comment.
[Review] - PowerPoint Loader
Review OS: Mac
Template Rule 가이드를 준수 하였는가: yes
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: yes
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: N/A
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: yes
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): yes
기타 의견:
- 최신 소스코드 반영과 라이브러리 반영이 인상적이었습니다. 꼼꼼하게 많은 분량 작성해주신 것 같습니다!!
[Review] - WebBaseLoader
Review OS: Mac
Template Rule 가이드를 준수 하였는가: yes
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: yes
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: N/A
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: yes
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): yes
기타 의견:
- 마지막 프록시 부분은 저도 테스트해보지 못했는데 전반적으로 꼼꼼하게 작성해주신 것 같습니다!
- 작은 의견이지만 템플릿 오타로 인해 Powerpoint loader, Webbase loader 모두 Table of Contents의 Environment 부분 오타가 있습니다!
- 마지막까지 정말 고생 많으셨습니다!!
Table of Contents typo fix
|
오타수정도 해주셨네요 확인했습니다!👍 |
저도 powerpoint 마지막에 이거 오류나네요 그리고 webbasedloader가 08번이 맞나요? |
- PPT 샘플 파일을 기존 파일로 변경하였습니다.
|
모두 리뷰해주셔서 감사합니다!! 마지막 셀은 파일이 잘못 들어가있어서 다시 변경했더니 작동하는 것 같습니다. |
[Title] PowerPoint-Loader
[Version] fix
[Language] ENG
[Packages] langchain-community, unstructured, markitdown
[Title] Webbase-Loader
[Version] initial
[Language] ENG
[Packages]langchain_community, markitdown