Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

architectyou
Copy link
Member

[Team] New Content Development Team1
[Title] HWP Loader
[Version] initial
[Language] ENG
[Packages] langchain-teddynote

Implemented HWP Loader & Sample document (법령정보센터 데이터를 영어로 번역)

@architectyou architectyou reopened this Jan 2, 2025
@architectyou architectyou changed the title [N-1] 06-Document Loader / 02-HWP-loader [N-1] 06-Document Loader / 03-HWP-loader Jan 2, 2025
@architectyou architectyou changed the title [N-1] 06-Document Loader / 03-HWP-loader [N-1] 06-Document Loader / 03-HWP-Loader Jan 2, 2025
Copy link
Contributor

@HarryKane11 HarryKane11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Review]

Review OS: Windows
Template Rule 가이드를 준수 하였는가: YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: No
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES

기타 의견:

  • Table of Contents에서 Load 부분 링크가 가장 윗 부분으로 앵커링되는 것 같습니다! 하위 섹션명인 "Loader"로 괄호 안 텍스트를 대체해주시면 될 것 같습니다.
  • Reference의 Query construction 부분은 어디에 참조되는지 궁금합니다. 혹시 참고한 부분이 없다면 지워도 되지 않을까 합니다!
  • 이외 나머지 코드 실행 오류나 패키지 호출은 이상 없습니다.

고생 많으셨습니다.

@architectyou
Copy link
Member Author

@HarryKane11 님께서 리뷰해주신 내용 반영하여 수정 완료하였습니다!

@suhyun0115
Copy link
Contributor

suhyun0115 commented Jan 2, 2025

[Review]

Review OS: Windows
Template Rule 가이드를 준수 하였는가: YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES

기타 의견:
고생하셨습니다!

Copy link
Contributor

@HarryKane11 HarryKane11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reference도 따로 찾아주셨네요. 고생하셨습니다!

@BAEM1N BAEM1N merged commit f18684b into LangChain-OpenTutorial:main Jan 2, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants