[N-1] 06-DocumentLoader / 05-Excel-Loader #77

forwardyoung · 2025-01-02T03:41:51Z

[Title] Excel Loader
[Version] fix
[Language] ENG
[Packages] unstructured, openpyxl

Implemented Excel loader tutorial with examples

Added UnstructuredExcelLoader and DataFrameLoader usage guide
Included code samples for raw text and structured data handling

Rename

Changed ipynb file name as 05-Excel-Loader.ipynb
Rename image files accroding to formatting conventions

Fix

increase recursion limit for UnstructuredExcelLoader

[Title] Excel File Loading in LangChain [Version] initial [Language] ENG [Packages] unstructured, openpyxl Implemented Excel loader tutorial with examples - Added UnstructuredExcelLoader and DataFrameLoader usage guide - Included code samples for raw text and structured data handling

[Title] Excel File Loading in LangChain [Version] initial [Language] ENG [Packages] unstructured, openpyxl Implemented Excel loader tutorial with examples - Added UnstructuredExcelLoader and DataFrameLoader usage guide - Included code samples for raw text and structured data handling Rename - Changed ipynb file name as

[Title] Excel File Loading in LangChain [Version] initial [Language] ENG [Packages] unstructured, openpyxl Implemented Excel loader tutorial with examples - Added UnstructuredExcelLoader and DataFrameLoader usage guide - Included code samples for raw text and structured data handling Rename - Rename image files accroding to formatting conventions

Peer Review 추가하여 반영하였습니다.

Environment 오타 수정하였습니다.

teddylee777

[Review]

Review OS(Windows/Mac/Linux): Mac
Template Rule 가이드를 준수 하였는가(YES/NO): YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): YES
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES
기타 의견:
꼼꼼히 작업해 주시느라 수고 많으셨습니다.
html 캡쳐본도 확인 하였습니다.

이미지만 가이드에 따라 전부 소문자 표기 다음에 작업해 주실때 그렇게 해주시면 되겠습니다!
감사합니다.
approval.

forwardyoung · 2025-01-02T14:45:28Z

[Review]

Review OS(Windows/Mac/Linux): Mac

Template Rule 가이드를 준수 하였는가(YES/NO): YES

Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES

이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): YES

import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES

모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES

기타 의견:
꼼꼼히 작업해 주시느라 수고 많으셨습니다.
html 캡쳐본도 확인 하였습니다.

이미지만 가이드에 따라 전부 소문자 표기 다음에 작업해 주실때 그렇게 해주시면 되겠습니다! 감사합니다. approval.

와! 제 pr 첫 리뷰 테디님께서 남겨주셨군요:)
감사합니다! 이미지 소문자 표기 꼭 참고하겠습니다.

jhboyo · 2025-01-03T01:28:22Z

[Review]

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

Recursion Limit가 작게 세팅 된 사용자를 위해 Limit 세팅이 필요해보이기도 합니다.
ex)
import sys
sys.setrecursionlimit(150000)

[Title] Excel File Loading in LangChain [Version] initial [Language] ENG [Packages] unstructured, openpyxl Fix: increase recursion limit for UnstructuredExcelLoader

forwardyoung · 2025-01-03T04:58:55Z

@jhboyo 리뷰 감사합니다!
말씀해주신 대로 Unstructured import 하는 코드에 recursion limit 늘려봤습니다!
확인 부탁드리겠습니다:)

[Title] Excel File Loading in LangChain [Version] clear [Language] ENG [Packages] unstructured, openpyxl Clear notebook output cells

[Title] Excel File Loading in LangChain [Version] chore [Language] ENG [Packages] unstructured, openpyxl chore: Modify peer name

jhboyo

sys.setrecursionlimit(10**6)
코드 추가 된 후 실행에 이상 없음을 확인하였습니다.

고생 많으셨습니다!

leebeanbin · 2025-01-03T05:51:48Z

[Review]

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

늦은 리뷰 죄송합니다. 그리고 리뷰부분에 Limit 10**6부분 확인 했고, 저도 이상 없었습니다.

leebeanbin

다시 한번 확인 후 문제 코드 실행에 문제 없어, 승인하겠습니다!

teddylee777 · 2025-01-03T07:34:10Z

@forwardyoung 님 안녕하세요?

지금 commit 된 파일에 .gitignore 를 포함한 튜토리얼 외적인 파일이 같이 commit 된 것 같습니다.
튜토리얼 (.ipynb) 파일과 data 파일 그리고 image 파일만 반영 부탁 드립니다.

파일 제외 하시고 다시 commit 만 해주시면 됩니다 (변경사항 revert)

[Title] Excel File Loading in LangChain [Version] chore [Language] ENG [Packages] unstructured, openpyxl chore: Remove unnecessary files

[Conflict Resolve] LangChain-OpenTutorial#77

forwardyoung added 2 commits January 2, 2025 00:00

forwardyoung requested review from jhboyo, teddylee777 and leebeanbin January 2, 2025 03:41

forwardyoung changed the title ~~[N-1] 06-DocumentLoader / 04-Excel-Loader~~ [N-1] 06-DocumentLoader / 05-Excel-Loader Jan 2, 2025

forwardyoung and others added 3 commits January 2, 2025 17:17

[Review] 06-DocumentLoader / 05-Excel-Loader

eb08120

Peer Review 추가하여 반영하였습니다.

[Review] 06-DocumentLoader / 05-Excel-Loader

b921a4a

Environment 오타 수정하였습니다.

teddylee777 previously approved these changes Jan 2, 2025

View reviewed changes

forwardyoung added 2 commits January 3, 2025 13:11

[Team] New Content Development Team1

5416d19

[Title] Excel File Loading in LangChain [Version] initial [Language] ENG [Packages] unstructured, openpyxl Fix: increase recursion limit for UnstructuredExcelLoader

[Team] New Content Development Team1

71d0259

[Title] Excel File Loading in LangChain [Version] initial [Language] ENG [Packages] unstructured, openpyxl Fix: increase recursion limit for UnstructuredExcelLoader

forwardyoung dismissed teddylee777’s stale review via 71d0259 January 3, 2025 04:56

forwardyoung added 2 commits January 3, 2025 14:09

[Team] New Content Development Team1

2811a5c

[Title] Excel File Loading in LangChain [Version] clear [Language] ENG [Packages] unstructured, openpyxl Clear notebook output cells

[Team] New Content Development Team1

2479c37

[Title] Excel File Loading in LangChain [Version] chore [Language] ENG [Packages] unstructured, openpyxl chore: Modify peer name

jhboyo previously approved these changes Jan 3, 2025

View reviewed changes

leebeanbin previously approved these changes Jan 3, 2025

View reviewed changes

[Team] New Content Development Team1

95c01ce

[Title] Excel File Loading in LangChain [Version] chore [Language] ENG [Packages] unstructured, openpyxl chore: Remove unnecessary files

forwardyoung dismissed stale reviews from leebeanbin and jhboyo via 95c01ce January 3, 2025 08:27

forwardyoung and others added 3 commits January 3, 2025 17:49

[Team] New Content Development Team1

7ee2014

[Title] Excel File Loading in LangChain [Version] chore [Language] ENG [Packages] unstructured, openpyxl chore: Remove unnecessary files

Merge branch 'main' into pr/77

b95ce75

[Conflict Resolve] LangChain-OpenTutorial#77

4eed09b

[Conflict Resolve] LangChain-OpenTutorial#77

teddylee777 merged commit ba1d421 into LangChain-OpenTutorial:main Jan 3, 2025
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[N-1] 06-DocumentLoader / 05-Excel-Loader #77

[N-1] 06-DocumentLoader / 05-Excel-Loader #77

Uh oh!

forwardyoung commented Jan 2, 2025 •

edited

Loading

Uh oh!

teddylee777 left a comment

Uh oh!

forwardyoung commented Jan 2, 2025

Uh oh!

jhboyo commented Jan 3, 2025

Uh oh!

forwardyoung commented Jan 3, 2025

Uh oh!

jhboyo left a comment

Uh oh!

leebeanbin commented Jan 3, 2025

Uh oh!

leebeanbin left a comment

Uh oh!

teddylee777 commented Jan 3, 2025

Uh oh!

Uh oh!

Uh oh!

[N-1] 06-DocumentLoader / 05-Excel-Loader #77

[N-1] 06-DocumentLoader / 05-Excel-Loader #77

Uh oh!

Conversation

forwardyoung commented Jan 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

teddylee777 left a comment

Choose a reason for hiding this comment

Uh oh!

forwardyoung commented Jan 2, 2025

Uh oh!

jhboyo commented Jan 3, 2025

Uh oh!

forwardyoung commented Jan 3, 2025

Uh oh!

jhboyo left a comment

Choose a reason for hiding this comment

Uh oh!

leebeanbin commented Jan 3, 2025

Uh oh!

leebeanbin left a comment

Choose a reason for hiding this comment

Uh oh!

teddylee777 commented Jan 3, 2025

Uh oh!

Uh oh!

Uh oh!

forwardyoung commented Jan 2, 2025 •

edited

Loading