-
Notifications
You must be signed in to change notification settings - Fork 282
[E-4] 05-Memory / 03-ConversationTokenBufferMemory #286
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
for PR proofreading
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.
이슈 확인하고 재작업하시느라 고생 많으십니다!
타 이슈 작업 중 확인한 사항인데,
Table of Contents
중
Environement Setup
-> Environment Setup
오타가 있어서 바꿔야 한다는 걸 뒤늦게 확인했네요.. 이 부분만 다시 해서 올려주시면 감사하겠습니다.
번거롭게 해드려서 죄송합니다 🙇
Environement > Environment
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.
수고하셨습니다!!
The merge-base changed after approval.
안녕하세요 @gyjong님, 모든 검토 및 2 approvals 이후 머지대기였던 PR에 변경사항이 생겨 알림 드립니다. 확인 후 Conflict resolve 요청드려요. 감사합니다 |
감사 합니다. 제가 빨리 확인하고 조치 취하도록 하겠습니다. |
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.
Conflict 발생한 건 직접 해결했는데 보니까 [Limiting Maximum Token Length to 50] 부분이 Table of Content에서만 변경되고 하단의 실제 heading 나와있는 부분에선 변경이 안 되었네요.
해당 부분만 해결해주시면 감사하겠습니다!
@chaeyoonyunakim 님 주말에도 알려 주셔서 감사 합니다. |
네 잘 해결된 것 같습니다! @BokyungisaGod 최종 approval 부탁드립니다~ |
4f01fcb
into
LangChain-OpenTutorial:main
Fixes Proofread: < 05-Memory / 03-ConversationTokenBufferMemory > #184
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.
(Optional) Related Issue: If this PR is linked to an issue, I have referenced the issue number in the PR message. (e.g., Fixes Update 01-PromptTemplate.ipynb #123)
❌ Do not include unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) or other people's code. If included, close the PR and create a new PR.