[N-2] 09-VectorStore / 06-Elasticsearch #352

namyoungkim · 2025-01-14T02:30:59Z

{PR Message - Write freely, remove this bracket if unnecessary}

Author Checklist

PR Title Format: I have confirmed that the PR title follows the correct format. (e.g., [N-2] 07-Text Splitter / 07-RecursiveCharacterTextSplitter)
Committed Files: I have ensured that no unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) are included. These files are not allowed.
close the PR #271

Review Template (Intial PR)

🖥️ OS: Win/Mac/Linux   
✅ Checklist      
 - [ ] **Template**: Tutorials follows the required template. 
 - [ ] **Table of Contents(TOC) Links**: All Table of Contents links work. ((Yes/No)
 - [ ] **Image**: Image filenames follow guidelines.
 - [ ] **Imports*: All import statements use the latest versions. Ensure "langchain-teddynote" is not used. 
 - [ ] **Code Execution**: Code runs without errors.
 - Comments: {Write freely, 한국어 기술 가능}

If no one reviews your PR within a few days, please @-mention one of teddylee777, musangk, BAEM1N

실수로 기존 PR을 닫아서 다시 커밋 및 PR을 생성했습니다.

…utorial

namyoungkim · 2025-01-14T02:35:43Z

@pupba @XaviereKU @sohyunwriter
광원님, 종호님, 소현님

안녕하세요
제가 실수로 #271 을 닫아 버려서 다시 PR을 열었습니다. 🙇🏻‍♂️

리뷰해주신 내용도 내일까지 반영하여 올리겠습니다~!

XaviereKU

🖥️ OS: Win
✅ Checklist

[] Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
[] Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments:
- Environmental setup 쪽에 ---로 줄을 그으셨는데 이게 템플릿에 부합하는지 확인이 필요할 것 같습니다. 그 외에 backtick(`) 역시 올바르게 사용됐는지 확인이 필요할 것 같습니다.
- 강조표시가 백틱( ) 으로 들어간게 있는 것 같습니다. 확인 부탁드립니다. ex) Upsert 섹션의 Let's perform an upsert operation fora single document.`
- ### 아래 설명이 들어가야 하는 것으로 알고 있습니다. 해당 cell에서 어떤 내용을 다룰지 간단하게라도 적어주시면 좋을 것 같습니다.
- PyTorch device auto selection 코드가 제공되어 좋습니다.
- set_env에 LANGCHAIN_API_KEY가 빠져서 추가하고 테스트 했습니다. 일부러 빼셨다면 코멘트 부탁드립니다.
- 이건 그냥 궁금한건데 일부러 OpenAIEmbedding을 안쓰신건지 궁금합니다.

수고하셨습니다. 감사합니다!

pupba

🖥️ OS: Win
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments: {Write freely, 한국어 기술 가능}
작성하시느라 고생하셨습니다!!
워낙 앞에 수정해야 할 부분 잘 말씀해주셔서 앞에 언급한 내용들 수정되었으면 좋겠습니다.
추가로 Mac이나 Linux에서는 상관 없지만 Window 환경에서 Python을 사용할 때
with open 구문을 사용 할 때, open 의 매개변수로 encoding="utf-8" 를 추가해주셔야 합니다. Window는 encoding 방식이 cp1252 이기 때문입니다.
튜토리얼 만드시느라 고생하셨습니다. 수정하시고 다시 Review 요청 주세요 :)

- Removed unnecessary "---" separator in Environmental setup section - Adjusted usage of "backtick(`)" and replaced excessive backticks with "**" for emphasis - Added descriptions under "###" headers for clarity - Included missing LANGCHAIN_API_KEY in set_env and tested functionality - Clarified usage of Hugging Face embedding over OpenAIEmbedding - Updated Windows-specific file handling with `encoding="utf-8"` and added line break handling

namyoungkim · 2025-01-16T04:11:51Z

@XaviereKU
상세한 리뷰 정말 감사합니다~!
리뷰 주신 내용 반영했습니다!🙇🏻‍♂️

리뷰1) Environmental setup 쪽에 ---로 줄을 그으셨는데 이게 템플릿에 부합하는지 확인이 필요할 것 같습니다.

답변) 첫 번째 파일을 만들고 검수 받을 때 이슈가 되지 않아서 동일한 포맷을 사용했습니다. 다만, 의견 주신 것처럼 "---"을 그을 필요가 없어서 제외했습니다!

리뷰2) 그 외에 backtick(`) 역시 올바르게 사용됐는지 확인이 필요할 것 같습니다.

강조표시가 백틱( ) 으로 들어간게 있는 것 같습니다.
확인 부탁드립니다. ex) Upsert 섹션의 Let's perform an upsert operation fora single document.`
- 답변) "backtick(`)", "**" 사용 후에는 공백 1칸을 적용하여 작성했습니다.
- 다만, "backtick(`)"을 많이 사용한 것 같아서, 문장 전체 강조 등은 "**"로 수정했습니다.

리뷰3) "###" 아래 설명이 들어가야 하는 것으로 알고 있습니다. 해당 cell에서 어떤 내용을 다룰지 간단하게라도 적어주시면 좋을 것 같습니다.

답변) 넵, 작성하였습니다~!

리뷰4) PyTorch device auto selection 코드가 제공되어 좋습니다.

답변) 감사합니다~!

리뷰5) set_env에 LANGCHAIN_API_KEY가 빠져서 추가하고 테스트 했습니다. 일부러 빼셨다면 코멘트 부탁드립니다.

답변) 실수로 빠트린게 맞습니다! 추가했습니다. 감사합니다.

리뷰6) 이건 그냥 궁금한건데 일부러 OpenAIEmbedding을 안쓰신건지 궁금합니다.

답변) 네, 허깅페이스 임베딩으로도 튜토리얼을 만들어두고 싶어서 OpenAIEmbedding을 사용하지 않았습니다.

namyoungkim · 2025-01-16T04:13:57Z

@pupba
리뷰해주셔서 감사합니다!
의견 주신 부분 수정했습니다. 확인부탁드려요~~!

리뷰7) 추가로 Mac이나 Linux에서는 상관 없지만 Window 환경에서 Python을 사용할 때, with open 구문을 사용 할 때, open 의 매개변수로 encoding="utf-8" 를 추가해주셔야 합니다. Window는 encoding 방식이 cp1252 이기 때문입니다.

답변) 좋을 리뷰 감사합니다. window를 고려하지 않았네요.
말씀주신 내용 반영했습니다. 추가로 줄바꿈 관련 코드도 운영체제마다 다를 것같아서 이부분도 추가했습니다~! 감사합니다!!

XaviereKU

🖥️ OS: Win/Mac/Linux
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments:
Advanced search에 있는 각 bulletpoints에 대해 어떤 식의 검색인지 설명이 있으면 좋을 거 같습니다.
Hybrid search의 경우 LangChain Elasticsearch 설명을 보면 strategy를 hybrid로 주는 것 같습니다. 현재 하신 방법은 hybrid라기보단 dense retrieval 앞에 filter를 두는 것 같은데 확인 부탁드립니다.

pupba

🖥️ OS: Win
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments: {Write freely, 한국어 기술 가능}
수정하시느라 고생하셨습니다!!
일단 제가 건의 드린건 정상적으로 수정해 주셨군요! 저는 일단 Approve 드리겠습니다.
다른 reviewer님이 올려주신 추가 수정안도 확인하시고 수정해주세요! 고생하셨습니다.

namyoungkim · 2025-01-16T09:48:41Z

@XaviereKU
종호님, 리뷰내용 반영했습니다.

(리뷰) Advanced search에 있는 각 bulletpoints에 대해 어떤 식의 검색인지 설명이 있으면 좋을 거 같습니다.
- (답변) 네, 각 search 별 간단한 설명을 추가했습니다.
(리뷰) Hybrid search의 경우 LangChain Elasticsearch 설명을 보면 strategy를 hybrid로 주는 것 같습니다. 현재 하신 방법은 hybrid라기보단 dense retrieval 앞에 filter를 두는 것 같은데 확인 부탁드립니다.
- (답변) 하이브리드 서치를 다양하게 정의 및 구현해서 사용하는 것으로 알고 있는데요.
- 간단한 사용 사례정도만 추가하려고, 텍스트 기반의 키워드 검색과 벡터 기반의 문맥적 유사성 검색을 결합한 하이브리드 서치를 튜토리얼로 포함했습니다.
- 엘라스틱서치는 기본적으로 키워드 검색을 지원해주기 때문에 바로 결합해서 사용했습니다.

확인부탁드립니다~!! 🙇🏻‍♂️

XaviereKU

🖥️ OS: Win
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
[] Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments: 수정 사항 확인 했습니다. 고생하셨습니다!

namyoungkim · 2025-01-17T01:04:21Z

@XaviereKU 종호님, 상세한 리뷰 감사합니다! 🙇🏻‍♂️

@pupba 광원님, approve 부탁드리겠습니다~!! 🙇🏻‍♂️

pupba

🖥️ OS: Win
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments: 수정하시느라 고생하셨습니다! 이상없습니다!

- Reduced text output for better readability - Added timeout configuration to prevent timeout errors

namyoungkim · 2025-01-18T14:07:21Z

@XaviereKU @pupba @sohyunwriter
출력문 수정하였습니다.
approve 한번 만 더 부탁드리겠습니다!! 🙇🏻‍♂️

pupba

🖥️ OS: Win
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments: 확인했습니다!

XaviereKU

🖥️ OS: Win
✅ Checklist

Template: Tutorials follows the required template.
Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
Image: Image filenames follow guidelines.
*Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
Code Execution: Code runs without errors.
Comments:

확인했습니다. 수고하셨습니다!

namyoungkim added 5 commits January 8, 2025 23:49

[N-2] 09-VectorStore / 06-Elasticsearch

79be70a

Notebook updated: all cells executed, HuggingFace embeddings applied

987b44f

Merge branch 'LangChain-OpenTutorial:main' into main

7a994c6

[N-2] 09-VectorStore / 06-Elasticsearch

bf79f9b

실수로 기존 PR을 닫아서 다시 커밋 및 PR을 생성했습니다.

Merge branch 'main' of https://github.com/namyoungkim/LangChain-OpenT…

516746e

…utorial

namyoungkim requested review from sohyunwriter, XaviereKU and pupba January 14, 2025 02:31

namyoungkim added 2 commits January 15, 2025 23:17

Merge branch 'LangChain-OpenTutorial:main' into main

9b7c47e

[N-2] 09-VectorStore / 06-Elasticsearch

7b12e66

XaviereKU requested changes Jan 15, 2025

View reviewed changes

pupba requested changes Jan 16, 2025

View reviewed changes

XaviereKU reviewed Jan 16, 2025

View reviewed changes

pupba previously approved these changes Jan 16, 2025

View reviewed changes

docs: add explanations for each search type in Advanced Search

9887959

namyoungkim dismissed pupba’s stale review via 9887959 January 16, 2025 09:43

XaviereKU previously approved these changes Jan 16, 2025

View reviewed changes

pupba previously approved these changes Jan 18, 2025

View reviewed changes

refactor: improve readability and handle timeout error

602617d

- Reduced text output for better readability - Added timeout configuration to prevent timeout errors

namyoungkim dismissed stale reviews from pupba and XaviereKU via 602617d January 18, 2025 13:13

pupba approved these changes Jan 18, 2025

View reviewed changes

XaviereKU approved these changes Jan 18, 2025

View reviewed changes

teddylee777 merged commit dbd7947 into LangChain-OpenTutorial:main Jan 18, 2025
1 of 2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[N-2] 09-VectorStore / 06-Elasticsearch #352

[N-2] 09-VectorStore / 06-Elasticsearch #352

Uh oh!

namyoungkim commented Jan 14, 2025

Uh oh!

namyoungkim commented Jan 14, 2025

Uh oh!

XaviereKU left a comment •

edited

Loading

Uh oh!

pupba left a comment

Uh oh!

namyoungkim commented Jan 16, 2025 •

edited

Loading

Uh oh!

namyoungkim commented Jan 16, 2025 •

edited

Loading

Uh oh!

XaviereKU left a comment

Uh oh!

pupba left a comment

Uh oh!

namyoungkim commented Jan 16, 2025 •

edited

Loading

Uh oh!

XaviereKU left a comment

Uh oh!

namyoungkim commented Jan 17, 2025

Uh oh!

pupba left a comment

Uh oh!

namyoungkim commented Jan 18, 2025

Uh oh!

pupba left a comment

Uh oh!

XaviereKU left a comment

Uh oh!

Uh oh!

Uh oh!

[N-2] 09-VectorStore / 06-Elasticsearch #352

[N-2] 09-VectorStore / 06-Elasticsearch #352

Uh oh!

Conversation

namyoungkim commented Jan 14, 2025

Author Checklist

Review Template (Intial PR)

Uh oh!

namyoungkim commented Jan 14, 2025

Uh oh!

XaviereKU left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pupba left a comment

Choose a reason for hiding this comment

Uh oh!

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

Uh oh!

Uh oh!

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

Uh oh!

Uh oh!

XaviereKU left a comment

Choose a reason for hiding this comment

Uh oh!

pupba left a comment

Choose a reason for hiding this comment

Uh oh!

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

Uh oh!

Uh oh!

XaviereKU left a comment

Choose a reason for hiding this comment

Uh oh!

namyoungkim commented Jan 17, 2025

Uh oh!

pupba left a comment

Choose a reason for hiding this comment

Uh oh!

namyoungkim commented Jan 18, 2025

Uh oh!

pupba left a comment

Choose a reason for hiding this comment

Uh oh!

XaviereKU left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

XaviereKU left a comment •

edited

Loading

namyoungkim commented Jan 16, 2025 •

edited

Loading

namyoungkim commented Jan 16, 2025 •

edited

Loading

namyoungkim commented Jan 16, 2025 •

edited

Loading