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

Skip to content

Conversation

brian604
Copy link
Contributor

Past URL

I have closed the PR: #365 and opened this a new one.
Have erased the new content that has been brought up (2025/01/18)
Review Template (After Proofread and Peer Reviews)
I have reflected all the changes, suggested by the PR reviews and issue reviews.

If no one reviews your PR within a few days, please @-mention one of teddylee777, musangk, BAEM1N

brian604 and others added 4 commits January 9, 2025 00:22
- Computer: Mac Air Mini (basemodel)
- Did not use LLM API, yet
- Initial Commit
----
Author Checklist
- [ x ] PR Title Format: I have confirmed that the PR title follows the correct format. (e.g., [N-2] 07-Text Splitter / 07-RecursiveCharacterTextSplitter)

- [ x ] Committed Files: I have ensured that no unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) are included. These files are not allowed.

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 (@)
1. LangChain-OpenTutorial#113
2. LangChain-OpenTutorial#299

All comments have been reflected.
@brian604 brian604 added the proofreading 번역/검수팀 제안사항 반영 label Jan 18, 2025
@BokyungisaGod
Copy link
Contributor

안녕하세요 @brian604 님, 수정사항이 조금 있어서 코멘트를 남기게 되었습니다.
1.
스크린샷 2025-01-19 오후 4 03 03
References 아래에 ---- 처리 해주세요!
ex)
스크린샷 2025-01-19 오후 4 03 56
2.
스크린샷 2025-01-19 오후 4 07 27
에러가 난 채로 올리지 말아주세요

스크린샷 2025-01-19 오후 4 08 58

반영해서 커밋 해주시면 approve 하겠습니다~!

@brian604
Copy link
Contributor Author

저 부분은 @teddylee777 님께서 에러에 대한 케이스에 대한 설명이어서요.. 오히려 놔두는게 이득이 될거 같습니다.

@brian604
Copy link
Contributor Author

pprint는 아직 langchain 패키지에 반영이 되지 않는것 같네요. 패키지 리퀘스트를 날리고 다시 PR을 날려야 할까요? @BokyungisaGod

@BokyungisaGod
Copy link
Contributor

앗 그런가요 ㅎㅎ 그러면 warning메세지는 따로 수정하지 않아도 됩니당.
+) pprint의 경우 패키지 리퀘스트 날린 후 PR을 다시 날리시는게 좋을 것 같습니다!

@brian604
Copy link
Contributor Author

pprint 같은 경우는 "Python" 자체에서 default로 제공하는 패키지이네요. 제가 pprintpp 를 다운로드해서 생겨나는 문제인것 같습니다 @BokyungisaGod

@BokyungisaGod
Copy link
Contributor

확인했습니다:)

@chaeyoonyunakim
Copy link
Contributor

@BokyungisaGod 님 approval 완료까지 요청드려요

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.

수고하셨습니다:)

@teddylee777 teddylee777 merged commit bf8163e into LangChain-OpenTutorial:main Jan 22, 2025
1 of 2 checks passed
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.

4 participants