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

Skip to content

Conversation

gyjong
Copy link
Contributor

@gyjong gyjong commented Jan 9, 2025

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.

@gyjong gyjong self-assigned this Jan 9, 2025
Copy link
Contributor

@ee-juni ee-juni left a 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
@gyjong
Copy link
Contributor Author

gyjong commented Jan 9, 2025

Table of Contents 내용 수정했습니다. 꼼꼼하게 봐 주셔서 너무 감사 합니다.
수고 하십시오.

@chaeyoonyunakim chaeyoonyunakim self-requested a review January 9, 2025 20:07
chaeyoonyunakim
chaeyoonyunakim previously approved these changes Jan 9, 2025
@chaeyoonyunakim chaeyoonyunakim added the proofreading 번역/검수팀 제안사항 반영 label Jan 9, 2025
ee-juni
ee-juni previously approved these changes Jan 10, 2025
Copy link
Contributor

@ee-juni ee-juni left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

BokyungisaGod
BokyungisaGod previously approved these changes Jan 10, 2025
@gyjong gyjong dismissed stale reviews from BokyungisaGod, ee-juni, and chaeyoonyunakim January 10, 2025 13:37

The merge-base changed after approval.

@chaeyoonyunakim
Copy link
Contributor

안녕하세요 @gyjong님, 모든 검토 및 2 approvals 이후 머지대기였던 PR에 변경사항이 생겨 알림 드립니다. 확인 후 Conflict resolve 요청드려요. 감사합니다

@gyjong
Copy link
Contributor Author

gyjong commented Jan 11, 2025

감사 합니다. 제가 빨리 확인하고 조치 취하도록 하겠습니다.

Copy link
Contributor

@ee-juni ee-juni left a 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 chaeyoonyunakim added the bug Something isn't working e.g., PR conflict label Jan 11, 2025
@gyjong
Copy link
Contributor Author

gyjong commented Jan 12, 2025

@chaeyoonyunakim 님 주말에도 알려 주셔서 감사 합니다.
다시 한번만 확인을 부탁 드립니다. Branch를 만들어서 수정 했는데요.
잘 반영이 되었나 걱정 입니다. 감사 합니다.

@ee-juni
Copy link
Contributor

ee-juni commented Jan 13, 2025

네 잘 해결된 것 같습니다! @BokyungisaGod 최종 approval 부탁드립니다~

@teddylee777 teddylee777 merged commit 4f01fcb into LangChain-OpenTutorial:main Jan 13, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working e.g., PR conflict proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants