Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

acho98
Copy link
Contributor

@acho98 acho98 commented Feb 17, 2025

{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-3] 17-LangGraph / 17-LongTermMemoryAgent)
  • 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 Proofread: 17 - LangGraph / 17 - LongTermMemoryAgent #697 )
  • ❌ 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.

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

@hong-seongmin 님, @BokyungisaGod 님 감사합니다.

@acho98 acho98 added the proofreading 번역/검수팀 제안사항 반영 label Feb 17, 2025
@acho98 acho98 changed the title Apply proofreading feedback post-merge Proofread: 17 - LangGraph / 17 - LongTermMemoryAgent #697 Feb 17, 2025
@hong-seongmin
Copy link
Contributor

현재 다시 확인해보니 소제목(###) 표시가 덜 되어 있는 것 같습니다.

Inspired by Research

The concept of long-term memory in agents is inspired by papers like MemGPT and is based on our own work. The agent extracts memories from chat interactions and stores them in a database.

Memory Representation

In this tutorial, "memory" is represented in two ways:

  • Text Information: A piece of text generated by the agent.
  • Structured Information: Knowledge about entities extracted by the agent, represented as (subject, predicate, object) knowledge triples.

Key Idea: Shared Memory Across Conversations

The key idea is that by saving memories, the agent persists information about users that is shared across multiple conversations (threads), which is different from memory of a single conversation that is already enabled by LangGraph's persistence.

만약 의도하신 바라면 그대로 comment를 남겨주시고 approve하겠습니다.

@acho98
Copy link
Contributor Author

acho98 commented Feb 17, 2025

현재 다시 확인해보니 소제목(###) 표시가 덜 되어 있는 것 같습니다.

Inspired by Research

The concept of long-term memory in agents is inspired by papers like MemGPT and is based on our own work. The agent extracts memories from chat interactions and stores them in a database.

Memory Representation

In this tutorial, "memory" is represented in two ways:

  • Text Information: A piece of text generated by the agent.
  • Structured Information: Knowledge about entities extracted by the agent, represented as (subject, predicate, object) knowledge triples.

Key Idea: Shared Memory Across Conversations

The key idea is that by saving memories, the agent persists information about users that is shared across multiple conversations (threads), which is different from memory of a single conversation that is already enabled by LangGraph's persistence.

만약 의도하신 바라면 그대로 comment를 남겨주시고 approve하겠습니다.

안녕하세요 @hong-seongmin
그 부분은 의도한 내용이라 소제목 표시하지 않았습니다

감사합니다 ~

hong-seongmin
hong-seongmin previously approved these changes Feb 18, 2025
Copy link
Contributor

@hong-seongmin hong-seongmin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

큰 오류를 발견하지 못하였습니다.
수고하셨습니다.

@BokyungisaGod
Copy link
Contributor

안녕하세요 @acho98 님, 수정사항이 있습니다.

스크린샷 2025-02-18 오후 1 44 47

References 밑에는 ---- 처리를 해주셔야 합니다.

이 부분만 수정해주시면 approve하겠습니다~

@BokyungisaGod
Copy link
Contributor

+) PR 양식이 따로 있는걸로 알고 있는데, 내용 수정해주세요~!

@acho98
Copy link
Contributor Author

acho98 commented Feb 18, 2025

안녕하세요 @acho98 님, 수정사항이 있습니다.

스크린샷 2025-02-18 오후 1 44 47 References 밑에는 ---- 처리를 해주셔야 합니다.

이 부분만 수정해주시면 approve하겠습니다~

수정하였습니다 ~

@acho98
Copy link
Contributor Author

acho98 commented Feb 18, 2025

+) PR 양식이 따로 있는걸로 알고 있는데, 내용 수정해주세요~!

수정하였습니다. 감사합니다 ~

@acho98 acho98 requested a review from hong-seongmin February 18, 2025 05:54
@acho98 acho98 linked an issue Feb 18, 2025 that may be closed by this pull request
8 tasks
BokyungisaGod
BokyungisaGod previously approved these changes Feb 18, 2025
Copy link
Contributor

@BokyungisaGod BokyungisaGod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다:)

hong-seongmin
hong-seongmin previously approved these changes Feb 18, 2025
Copy link
Contributor

@hong-seongmin hong-seongmin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다.

@teddylee777
Copy link
Contributor

@BAEM1N 님 이거 conflict resolve 한 번 봐주실 수 있어요?

@teddylee777 teddylee777 requested a review from BAEM1N March 3, 2025 10:32
@BAEM1N
Copy link
Contributor

BAEM1N commented Mar 3, 2025

@acho98 @teddylee777

제가 Push 권한이 없어서 코멘트로 대체하겠습니다.

  1. 로컬에서 새로운 브랜치 acho98-byoon_pr_0217을 생성해주시고 생성된 브랜치로 변경사항을 가져와주세요.
git checkout -b acho98-byoon_pr_0217 main
git pull https://github.com/acho98/LangChain-OpenTutorial.git byoon_pr_0217
  1. 변경사항을 main으로 병합해주세요.
git checkout main
git merge --no-ff acho98-byoon_pr_0217
git push origin main

위 두과정을 선행해주시고 conflicts이 풀리는지 확인해봐야할거같습니다!

@acho98
Copy link
Contributor Author

acho98 commented Mar 3, 2025

@acho98 @teddylee777

제가 Push 권한이 없어서 코멘트로 대체하겠습니다.

  1. 로컬에서 새로운 브랜치 acho98-byoon_pr_0217을 생성해주시고 생성된 브랜치로 변경사항을 가져와주세요.
git checkout -b acho98-byoon_pr_0217 main
git pull https://github.com/acho98/LangChain-OpenTutorial.git byoon_pr_0217
  1. 변경사항을 main으로 병합해주세요.
git checkout main
git merge --no-ff acho98-byoon_pr_0217
git push origin main

위 두과정을 선행해주시고 conflicts이 풀리는지 확인해봐야할거같습니다!

@BAEM1N 님께
코멘트 주신 1~2번 사항 모두 수행하였습니다.

@BAEM1N
Copy link
Contributor

BAEM1N commented Mar 5, 2025

@acho98 @teddylee777
제가 Push 권한이 없어서 코멘트로 대체하겠습니다.

  1. 로컬에서 새로운 브랜치 acho98-byoon_pr_0217을 생성해주시고 생성된 브랜치로 변경사항을 가져와주세요.
git checkout -b acho98-byoon_pr_0217 main
git pull https://github.com/acho98/LangChain-OpenTutorial.git byoon_pr_0217
  1. 변경사항을 main으로 병합해주세요.
git checkout main
git merge --no-ff acho98-byoon_pr_0217
git push origin main

위 두과정을 선행해주시고 conflicts이 풀리는지 확인해봐야할거같습니다!

@BAEM1N 님께 코멘트 주신 1~2번 사항 모두 수행하였습니다.

git checkout byoon_pr_0217

# 메인 브랜치의 최신 내용을 가져온 후 rebase
git pull origin main
git rebase main

위 명령어를 실행해 현재 PR을 올린 브랜치로 이동한 다음, main 브랜치의 내용을 가져와 rebase를 진행해 주세요. 이 과정에서 충돌이 발생하면 직접 충돌을 해결해 주시면 됩니다. (대부분의 경우 메인 브랜치의 내용을 우선하거나, 또는 byoon_pr_0217 브랜치 내용을 우선하는 방식으로 해결하면 됩니다.)

이후 아래 명령어를 실행하여 현재 PR에 올린 커밋 내용이 제대로 남는지 확인해 주세요.

# 충돌 해결 후 아래 명령어 입력
git rebase --continue

# 최종적으로 push (강제 푸시 필요할 수 있음)
git push acho98 byoon_pr_0217 --force-with-lease

현재 충돌이 발생한 이유는, PR은 byoon_pr_0217 브랜치로 올렸는데 작업이 main 브랜치에도 반영되어 브랜치 간 차이가 발생했기 때문으로 추측됩니다.

@acho98
Copy link
Contributor Author

acho98 commented Mar 6, 2025

@BAEM1N@teddylee777
@BAEM1N 님께서 제안해주신 방법으로 해보고 업데이트 올리겠습니다.
여전히 해결이 안되면 새로 PR 올리도록 하겠습니다

IHAGI-c and others added 4 commits March 7, 2025 08:04
* create weaviate_vectordb.py

* commit

* add create_collection func

* fix delete, search func

* notebook fix

* upsert fix

* search fix

* delete fix

* all run

* delete base.py

* review fix
* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Add initial MongoDBAtlas class

* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Add initial MongoDBAtlasDocumentManager
- Modify MongoDBAtlas method signatures

* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Add load and split data in MongoDBAtlasDocumentManager

* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Modify to use mongodb_atlas.py method

* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Add upsert, upsert_parallel, search, delete
- Modify document type

* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Modify tutorial explanation
- Add docstring in MongoDBAtlas

* [N-2] 09-VectorStore / 07-MongoDB-Atlas
- Modify tutorial explanation
- Add docstring in MongoDBAtlasDocumentManager
* feat: add elasticsearch_interface.py file

* Update Elasticsearch Document manager and improve document handling.

* fix: elasticsearch.py 로 파일명 수정

* fix: elasticsearch.py 로 파일명 수정

* fix: document_id 목록을 가져올 때, size를 설정할 수 있도록 변경

* fix: utils/elasticsearch.py 에 맞춰 튜토리얼 재작성

* docs: 기본 검색일 때, BM25 알고리즘으로 검색한다는 설명 추가
@acho98 acho98 dismissed stale reviews from hong-seongmin and BokyungisaGod via b9a48ad March 6, 2025 23:10
@acho98
Copy link
Contributor Author

acho98 commented Mar 6, 2025

@BAEM1N 님, @teddylee777
브랜치간 차이로 인한 파일들이 모두 커밋되었습니다(110개 이상)
새로 PR 올리도록 하겠습니다.
새로운 PR 올린 후 여기에 업데이트 해놓겠습니다.

@acho98
Copy link
Contributor Author

acho98 commented Mar 6, 2025

@BAEM1N 님, @teddylee777

신규 PR 올렸습니다. (링크 #844)
리뷰어는 기민님, 테디님으로 임의 할당 해놓았습니다.

감사합니다.

@teddylee777
Copy link
Contributor

신규 PR 오픈으로 인하여 close 하겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proofread: 17 - LangGraph / 17 - LongTermMemoryAgent