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

Skip to content

Conversation

choincnp
Copy link
Contributor

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.

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

@jeongkpa
Copy link
Contributor

@choincnp 님 안녕하세요 에러가 납니다 (WIN) 환경

image


Install required packages

from langchain_opentutorial import package

package.install(
[
"langsmith",
"langchain",
"langchain_core",
"langchain_openai",
"langchain_ollama",
"pymuPDF",
"fitz",
"frontend"
],
verbose=False,
upgrade=False,
)


이 부분도 설치해야하는 패키지가 누락된것 같습니다.

확인부탁드립니다!

Copy link
Contributor

@jeongkpa jeongkpa left a comment

Choose a reason for hiding this comment

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

리뷰 남긴 부분 수정 부탁드립니다!

@choincnp
Copy link
Contributor Author

@jeongkpa 안녕하세요, 혹시 llama 3.2:1b 돌리시면서 진행하신것 맞으실까요?

@jeongkpa
Copy link
Contributor

@jeongkpa 안녕하세요, 혹시 llama 3.2:1b 돌리시면서 진행하신것 맞으실까요?

@choincnp 안녕하세요, 회사컴에서 집으로 와서 WIN 환경에서 돌려보니 (올라마 설치돼있음) 잘 돌아가네요..!! 죄송합니다!

Copy link
Contributor

@jeongkpa jeongkpa left a comment

Choose a reason for hiding this comment

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

🖥️ OS: Win
✅ 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.

@teddylee777 teddylee777 merged commit 5ffd619 into LangChain-OpenTutorial:main Jan 18, 2025
1 of 2 checks passed
@sohyunwriter sohyunwriter added the docs tutorial label Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs tutorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants