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

Skip to content

Conversation

ppakyeah
Copy link
Contributor

@ppakyeah ppakyeah commented Jan 2, 2025

[Team] New content development teams 1
[Title] DocumentLoader/ PDFLoader
[Version] initial
[Language] ENG
[Packages] langchain_community, pypdf, rapidocr-onnxruntime, pymupdf, unstructured[pdf]

LangChain PDF Loader Tutorial

  • AutoRag team experiments appear to be Korean-based and have been removed
  • Change to the latest LangChain Document URL
  • Added descriptions and reference URLs for each library

@ppakyeah ppakyeah self-assigned this Jan 2, 2025
geminii01
geminii01 previously approved these changes Jan 2, 2025
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. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: N/A
  5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
  6. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES
  7. 기타 의견: 고생 많으셨습니다!!

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.

[Review]

  1. Review OS: Windows 11

  2. Template Rule 준수 여부: 예

  3. Table of Contents 링크 기능성: 예

  4. 이미지 파일명 규칙: 해당 없음

  • 튜토리얼에 별도의 이미지 파일이 포함되어 있지 않습니다.
  1. Import 구문 최신 버전 준수: 예
  • 최신 langchain_community 패키지를 사용하고 있습니다.
  1. 코드 동작 상태:
  • 대부분의 코드가 정상 작동하나, PyPDFium2Loader 사용 시 경고 메시지가 발생합니다.
  • 이 경고 메시지는 라이브러리의 내부 동작과 관련된 것으로, 코드의 실제 기능에는 영향을 미치지 않습니다.
  1. 추가 의견:
    PyPDFium2Loader 섹션에 다음과 같은 노트를 추가할 것을 제안합니다:
**Note**: When using PyPDFium2Loader, you may notice warning messages related to get_text_range(). 
These warnings are part of the library's internal operations and do not affect the PDF processing 
functionality. You can safely proceed with the tutorial despite these warnings, as they are 
a normal part of the development environment and do not impact the learning objectives.

요런 설명을 추가하면 사용자는 경고 메시지의 성격을 이해하고 학습에 집중할 수 있겠죠. 또 실제 개발 환경에서 흔히 발생할 수 있는 상황을 자연스럽게 경험할 수 있는 기회가 될 거에요. 그렇게되면 우리가 만드는 튜토리얼의 교육적 가치를 유지하면서도 불필요한 코드 복잡성을 피할 수 있는 적절한 해결책이 될 것같습니다!

PyPDFium2Loader guide comment 추가
PyPDFium2Loader warning output 제거
@teddylee777 teddylee777 merged commit 919307c into LangChain-OpenTutorial:main Jan 4, 2025
3 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