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

Skip to content

Conversation

Taylor0819
Copy link
Contributor

[Team] New Content Development Team 1
[Title] UpstageDocumentParseLoader
[Version] Initial
[Language] ENG
[Packages] langchain-upstage, langsmith

Initial tutorial for UpstageDocumentParseLoader using langchain-upstage

[Team] New Content Development Team 1
[Title] UpstageDocumentParseLoader
[Version] Initial
[Language] ENG
[Packages] langchain-upstage, langsmith

Initial tutorial for UpstageDocumentParseLoader using langchain-upstage
@Taylor0819 Taylor0819 changed the title Create 12.UpstageDocumentParseLoader.ipynb [N-1] 06-Document Loader / 12-UpstageDocumentParseLoader Jan 1, 2025
Copy link
Contributor

@back2zion back2zion left a comment

Choose a reason for hiding this comment

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

고생하셨네요. 살펴보니 문서는 잘 구성되었는데, 목차의 오타 수정과 환경 설정 부분의 보완이 필요해 보입니다. 1월 1일 새해 첫날 고생하셨습니다!

[Team] New Content Development Team 1
[Title] UpstageDocumentParseLoader
[Version] After Peer review1
[Language] ENG
[Packages] langchain-upstage, langsmith

Changes applied after peer review: incorporated API key issuance process, corrected typos, and added explanation for data download path.
back2zion
back2zion previously approved these changes Jan 1, 2025
Copy link
Contributor

@back2zion back2zion left a comment

Choose a reason for hiding this comment

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

밤 늦은 시간까지 고생하셨습니다!

[Team] New Content Development Team 1
[Title] UpstageDocumentParseLoader
[Version] after Peer review1-2
[Language] ENG
[Packages] langchain-upstage, langsmith

Convert the comments from Korean to English.
Copy link
Member

@geminii01 geminii01 left a comment

Choose a reason for hiding this comment

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

[Review]

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

API Key 발급 방법, 데이터 다운로드 방법 모두 잘 안내되어 있습니다.
고생하셨습니다👍👍

@teddylee777 teddylee777 merged commit d44f825 into LangChain-OpenTutorial:main Jan 2, 2025
1 of 3 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