-
Notifications
You must be signed in to change notification settings - Fork 282
LCEL add memory 추가 #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
[Title] LCEL add memory [Version] initial [Language] ENG [Packages] langchain / langchain_core / langchain_openai LCEL을 활용하여 chain에 memory를 추가하는 자료에 대해 생성하였습니다.
[Title] LCEL add memory detail 추가 [Version] revision [Language] ENG [Packages] langchain / langchain_core / langchain_openai - 김성철 님의 작업을 벤치마킹하여 `load_dotenv` 설명과 `ConversationBufferMemory` 설명을 보완하였습니다. - 개인적으로 작업하면서 개념이 헷갈렸던 `RunnablePassthrough`와 `RunnableLambda`에 대한 설명을 보완하였습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코랩에서 다 테스트 잘 하였습니다. 해적 스타일 이군요 ^^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Review]
- Review OS(Windows/Mac/Linux): Linux
- Template Rule 가이드를 준수 하였는가(YES/NO): Yes
- Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): No
- Extra: Migrating ConversationChainBufferMemory 가 제대로 동작하지 않습니다. 업데이트가 필요합니다.
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): Yes (이미지 없음)
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): 최신 버전
- 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): Yes
- 기타 의견: 해적 말투 예시가 너무 좋은 아이디어인 것 같습니다. 수고하셨습니다😄
[Title] LCEL add memory Table of Contents 수정 [Version] bugfix [Language] ENG [Packages] langchain / langchain_core / langchain_openai Table of Contents에 매칭되지 않는 소제목 링크를 제대로 작동하게끔 수정해 주었습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Review]
- Review OS(Windows/Mac/Linux): win
- Template Rule 가이드를 준수 하였는가(YES/NO): yes
- Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): yes
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): yes
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): yes
- 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요):
- 기타 의견:
[Title] LCEL add memory 추가
[Version] initial
[Language] ENG
[Packages] langchain / langchain_core / langchain_openai
Langchain tutorial 05-Memory
영어 번역본 생성하였습니다.