ConversationEntityMemory by ulysyszh · Pull Request #76 · LangChain-OpenTutorial/LangChain-OpenTutorial

ulysyszh · 2025-01-02T00:48:16Z

[Title] ConversationEntityMemory
[Version] initial
[Language] ENG
[Package] langchain, langchain-community, langchain-openai

ConversationEntityMemory Tutorial Implementation

[Title] 04_ConversationEntityMemory [Version] initial [Language] ENG [Packages] langchain langchain-community langchain-opentutorial

[Title] ConversationEntityMemory [Version] initial [Language] ENG [Package] langchain, langchain-community, langchain-openai ConversationEntityMemory Tutorial Implementation

sungchul2

[Review]

Review OS(Windows/Mac/Linux): Linux
Template Rule 가이드를 준수 하였는가(YES/NO): Yes
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): No

Example, Retrieving Entity Memory가 아래 내용에 없어서 실행이 안됩니다. 업데이트가 필요합니다.

이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): Yes (이미지 없음)
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO):
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요):

ConversationChain이 1.0에서 deprecated 예정이라는 warning이 뜨네요. 추후 RunnableWithMessageHistory를 사용하는 방식으로 수정되면 좋을 것 같습니다.

기타 의견:

예시가 Online Bank Account Opening Conversation에 대한 내용이 아닌 것 같습니다. 예시가 추가되던지, 설명이 수정되면 좋을 것 같습니다.
Environment Setup에서 사용될 패키지와 예제에서 사용될 패키지를 구분하면 좋을 것 같습니다. 현재는 모든 패키지가 Environment Setup에서 import되고 있습니다.

gyjong

[Review]

Review OS : Mac
Template Rule 가이드를 준수 하였는가 : YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가? : YES
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? : N/A(이미지 없음)
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? : YES
모든 코드가 동작에 오류 없이 동작하는가? : YES
기타 의견: 입력한 코멘트 사항 확인을 부탁 드립니다.

gyjong · 2025-01-02T05:35:14Z

+        "id": "Pkf5OMA6kJax"
+      },
+      "source": [
+        "# ConversationBufferWindowMemory\n",


제목을 'ConversationEntityMemory'으로 해야 할것 같습니다.
수고 하십시오.

gyjong · 2025-01-02T05:57:32Z

+      "source": [
+        "from langchain.prompts import PromptTemplate\n",
+        "\n",
+        "entity_memory_conversation_template = PromptTemplate(\n",


from langchain.memory.prompt import ENTITY_MEMORY_CONVERSATION_TEMPLATE 의 내용을 사용하는 방법도 좋을것 같습니다. 별도로 작성하신 entity_memory_conversation_template 내용도 간결하고 좋은것 같습니다.

[Title] ConversationEntityMemory [Version] Enhance: Fix error [Language] ENG [Package] langchain, langchain-community, langchain-openai ConversationEntityMemory Tutorial Implementation

Code Improvements & Error fixes reflecting feedback

ulysyszh · 2025-01-02T14:39:58Z

두분 너무 감사합니다~ 알려주신 것을 반영해봤습니다.

피드백 주신 내용들과 문서 내 바로가기 수정했습니다. 감사합니다~

ulysyszh · 2025-01-02T23:19:56Z

좋은 아침입니다. 테이블 링크 수정해서 한번 더 올려봅니다. 감사합니다~

gyjong

[Review]

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

from langchain_community.chat_models import ChatOpenAI 부분 확인 부탁 드립니다.
llm 선언 부분 확인을 부탁 드립니다.
감사 합니다.

gyjong · 2025-01-02T23:52:11Z

+      },
+      "outputs": [],
+      "source": [
+        "llm = ChatOpenAI(temperature=0)\n",


141의 openai = ChatOpenAI(model="gpt-4o-mini", temperature=0) 대신 여기에 llm = ChatOpenAI(model="gpt-4o-mini", temperature=0) 이렇게 적용하면 명확할것 같습니다. 사용하고 싶은신 모델에 따라서 model="gpt-4o-mini" 부분 수정하면 될것 같습니다.

sungchul2

[Review]

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

블록들 위치 및 내용을 수정해주시면 좋을 것 같습니다.

Co-authored-by: Sungchul Kim <sungchul7039@gmail.com>

피드백 주신 내용을 바탕으로 표현을 명확하게 하고 코드블럭 순서를 바꾸었습니다~ 감사합니다!

ulysyszh · 2025-01-03T22:42:18Z

피드백 참고해서 한번 더 수정했습니다~ 감사합니다

gyjong

[Review]

Review OS : Mac
Template Rule 가이드를 준수 하였는가 : YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가? : YES
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? : N/A(이미지 없음)
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? : YES
모든 코드가 동작에 오류 없이 동작하는가? : YES
기타 의견: 변경사항 확인 하였습니다. 수고 하셨습니다.

sungchul2

[Review]

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

gyjong

[Review Checklist]

Review OS:
- Windows / Mac / Linux (Mac)
Template Rule 준수 여부:
- Template Rule 가이드를 준수하였는가? (Check 표기)
Table of Contents 링크 확인:
- Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
이미지 파일명 검토:
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
최신 import 방식 사용 여부:
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
코드 동작 확인:
- 모든 코드가 오류 없이 동작하는가? (Check 표기)
- Warning 발생 시 코멘트에 적어주세요.
제출 파일 확인:
- 꼭 필요한 파일들만 제출되었는가?
  - 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
기타 의견:
고생 많으셨습니다.

ulysyszh added 7 commits January 2, 2025 05:48

Colab을 통해 생성됨

e4165e5

Delete 04_ConversationEntityMemory.ipynb

0e7bc12

Add files via upload

ec89191

Update 04_ConversationEntityMemory.ipynb

25b9a3b

Update 04_ConversationEntityMemory.ipynb

1c7a462

[Title] 04_ConversationEntityMemory [Version] initial [Language] ENG [Packages] langchain langchain-community langchain-opentutorial

Delete 05-Memory/04_ConversationEntityMemory.ipynb

1296622

Add files via upload

1e5b46a

[Title] ConversationEntityMemory [Version] initial [Language] ENG [Package] langchain, langchain-community, langchain-openai ConversationEntityMemory Tutorial Implementation

ulysyszh requested review from gyjong and sungchul2 January 2, 2025 00:48

sungchul2 reviewed Jan 2, 2025

View reviewed changes

Comment thread 05-Memory/04_ConversationEntityMemory.ipynb

Comment thread 05-Memory/04_ConversationEntityMemory.ipynb Outdated

Comment thread 05-Memory/04_ConversationEntityMemory.ipynb Outdated

Comment thread 05-Memory/04_ConversationEntityMemory.ipynb Outdated

sungchul2 reviewed Jan 2, 2025

View reviewed changes

Comment thread 05-Memory/04_ConversationEntityMemory.ipynb Outdated

sungchul2 reviewed Jan 2, 2025

View reviewed changes

Comment thread 05-Memory/04_ConversationEntityMemory.ipynb Outdated

gyjong requested changes Jan 2, 2025

View reviewed changes

ulysyszh added 7 commits January 2, 2025 17:17

Create test.ipynb

aaa8ecd

Update 04_ConversationEntityMemory.ipynb

0519174

Update 04_ConversationEntityMemory.ipynb

8fd7c7a

Delete 05-Memory/04_ConversationEntityMemory.ipynb

1378d2d

Add files via upload

92ec781

[Title] ConversationEntityMemory [Version] Enhance: Fix error [Language] ENG [Package] langchain, langchain-community, langchain-openai ConversationEntityMemory Tutorial Implementation

Delete 05-Memory/04_ConversationEntityMemory.ipynb

aa8ceeb

Add files via upload

f7e2ccf

Code Improvements & Error fixes reflecting feedback

ulysyszh requested review from gyjong and sungchul2 January 2, 2025 14:38

ulysyszh added 2 commits January 3, 2025 08:01

Delete 05-Memory/04_ConversationEntityMemory.ipynb

aaa6780

Add files via upload

0153eae

피드백 주신 내용들과 문서 내 바로가기 수정했습니다. 감사합니다~

gyjong requested changes Jan 2, 2025

View reviewed changes

sungchul2 reviewed Jan 3, 2025

View reviewed changes

ulysyszh and others added 2 commits January 4, 2025 05:41

Update 05-Memory/04_ConversationEntityMemory.ipynb

913f33c

Co-authored-by: Sungchul Kim <sungchul7039@gmail.com>

Delete 05-Memory/04_ConversationEntityMemory.ipynb

828b0b1

Add files via upload

e8701c4

피드백 주신 내용을 바탕으로 표현을 명확하게 하고 코드블럭 순서를 바꾸었습니다~ 감사합니다!

ulysyszh requested review from gyjong and sungchul2 January 3, 2025 22:42

gyjong previously approved these changes Jan 4, 2025

View reviewed changes

Update 04_ConversationEntityMemory.ipynb

c142ac9

ulysyszh dismissed gyjong’s stale review via c142ac9 January 4, 2025 07:46

ulysyszh added 2 commits January 4, 2025 16:50

Delete test.ipynb

41b751d

Update 04_ConversationEntityMemory.ipynb

a9f3d70

sungchul2 approved these changes Jan 4, 2025

View reviewed changes

teddylee777 self-requested a review January 4, 2025 11:09

teddylee777 merged commit 422102b into LangChain-OpenTutorial:main Jan 4, 2025

gyjong reviewed Jan 7, 2025

View reviewed changes

Conversation

ulysyszh commented Jan 2, 2025

Uh oh!

sungchul2 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gyjong left a comment

Choose a reason for hiding this comment

Uh oh!

gyjong Jan 2, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

gyjong Jan 2, 2025

Choose a reason for hiding this comment

Uh oh!

ulysyszh commented Jan 2, 2025

Uh oh!

ulysyszh commented Jan 2, 2025

Uh oh!

gyjong left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

gyjong Jan 2, 2025

Choose a reason for hiding this comment

Uh oh!

sungchul2 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ulysyszh commented Jan 3, 2025

Uh oh!

gyjong left a comment

Choose a reason for hiding this comment

Uh oh!

sungchul2 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!

gyjong left a comment

Choose a reason for hiding this comment

[Review Checklist]

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

sungchul2 left a comment •

edited

Loading