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

Skip to content

Conversation

forwardyoung
Copy link
Contributor

{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-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

@ppakyeah
Copy link
Contributor

ppakyeah commented Jan 16, 2025

πŸ–₯️ 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, ν•œκ΅­μ–΄ 기술 κ°€λŠ₯}
    • ν˜„μž¬λŠ” μ½”λ“œλ§Œ μž‘μ„±λ˜μ–΄ μžˆλŠ”λ° 반볡 평가에 λŒ€ν•œ μ„€λͺ… 및 κ°„λž΅ν•œ 라마 μ„€μΉ˜ κ°€μ΄λ“œλΌμΈ λ“±μ˜ ν…μŠ€νŠΈκ°€ μΆ”κ°€λ˜λ©΄ 더 쒋은 νŠœν† λ¦¬μ–Όμ΄ 될 것 κ°™μŠ΅λ‹ˆλ‹€!
    • overview에 λ„£μ–΄μ£Όμ‹  링크 References에도 μΆ”κ°€ν•˜λŠ”κ²Œ μ–΄λ–¨κΉŒ 의견 λ“œλ¦½λ‹ˆλ‹€.
    • 이미지 파일λͺ… κ°€μ΄λ“œλΌμΈμ— 맞게 μˆ˜μ •μ΄ ν•„μš”ν•©λ‹ˆλ‹€.
  • 이미지 파일λͺ…
    • λͺ¨λ‘ 영문 μ†Œλ¬Έμž 둜 μž‘μ„±
    • 곡백이 μ—†μ–΄μ•Ό ν•˜λ©° 곡백 λŒ€μ‹  - ν•˜μ΄ν”ˆμœΌλ‘œ λŒ€μ²΄
    • jupyter notebook 파일λͺ… + 이미지 제λͺ© + ν•„μš”μ‹œ 숫자 (01, 02, 03, ...)

@forwardyoung
Copy link
Contributor Author

@ppakyeah 리뷰 κ°μ‚¬ν•©λ‹ˆλ‹€! νŠœν† λ¦¬μ–Ό λ³΄μ™„ν•˜κ³  μ•Œλ €λ“œλ¦¬κ² μŠ΅λ‹ˆλ‹€.

Copy link
Contributor

@sungchul2 sungchul2 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/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.
    • λ§ˆμ§€λ§‰ μ…€μ—μ„œ ChatOllama둜 평가 쀑 μ•„λž˜μ™€ 같은 μ—λŸ¬κ°€ λ°œμƒν•˜κ³  μžˆμŠ΅λ‹ˆλ‹€. 확인해본 κ²°κ³Ό, run이 {"output": None} μ΄μ–΄μ„œ λ°œμƒν•˜κ³  μžˆμŠ΅λ‹ˆλ‹€. ν˜Ήμ‹œ μ €λ§Œ κ·ΈλŸ°κ±΄μ§€, 더블체크 λΆ€νƒλ“œλ¦½λ‹ˆλ‹€.
...
ollama._types.ResponseError: model 'llama3.2' not found, try pulling it first
Error running evaluator <DynamicRunEvaluator evaluate> on run cf34c270-2371-4e9c-bec6-9a3050491781: KeyError('answer')
Traceback (most recent call last):
  File "/home/sungchul/.pyenv/versions/langchain_tutorial_1/lib/python3.11/site-packages/langsmith/evaluation/_runner.py", line 1573, in _run_evaluators
    evaluator_response = evaluator.evaluate_run(
                         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/sungchul/.pyenv/versions/langchain_tutorial_1/lib/python3.11/site-packages/langsmith/evaluation/evaluator.py", line 331, in evaluate_run
    result = self.func(
             ^^^^^^^^^^
  File "/home/sungchul/.pyenv/versions/langchain_tutorial_1/lib/python3.11/site-packages/langsmith/run_helpers.py", line 617, in wrapper
    raise e
  File "/home/sungchul/.pyenv/versions/langchain_tutorial_1/lib/python3.11/site-packages/langsmith/run_helpers.py", line 614, in wrapper
    function_result = run_container["context"].run(func, *args, **kwargs)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/sungchul/.pyenv/versions/langchain_tutorial_1/lib/python3.11/site-packages/langsmith/evaluation/integrations/_langchain.py", line 258, in evaluate
    else self._prepare_data(run, example)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/tmp/ipykernel_1064808/2296627391.py", line 8, in <lambda>
    "prediction": run.outputs["answer"],
                  ~~~~~~~~~~~^^^^^^^^^^
KeyError: 'answer'
  • Comments: {Write freely, ν•œκ΅­μ–΄ 기술 κ°€λŠ₯}

@forwardyoung
Copy link
Contributor Author

@rlatjcj 리뷰 κ°μ‚¬ν•©λ‹ˆλ‹€. μ—λŸ¬ λ‚˜λŠ” λΆ€λΆ„ μˆ˜μ •ν•΄μ„œ μ˜¬λ¦¬κ² μŠ΅λ‹ˆλ‹€.

@forwardyoung
Copy link
Contributor Author

@ppakyeah 이미지 파일λͺ… μˆ˜μ • 및 νŠœν† λ¦¬μ–Ό λ‚΄μš© 보완 ν•˜μ˜€μŠ΅λ‹ˆλ‹€!
@rlatjcj νŒ¨ν‚€μ§€ μΆ”κ°€ν•˜κ³  μ—λŸ¬ λ°œμƒν•˜λŠ” λΆ€λΆ„ μˆ˜μ •ν–ˆμœΌλ©°, λ§ˆμ§€λ§‰ μ…€ 문제 없이 좜λ ₯λ˜λŠ” 것 ν™•μΈν•˜μ˜€μŠ΅λ‹ˆλ‹€!

Copy link
Contributor

@sungchul2 sungchul2 left a comment

Choose a reason for hiding this comment

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

πŸ–₯️ OS: 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, ν•œκ΅­μ–΄ 기술 κ°€λŠ₯}

@forwardyoung
Copy link
Contributor Author

@ppakyeah 말씀해주신 μ„Ήμ…˜μ— 컨텐츠 μΆ”κ°€ν•˜μ˜€μŠ΅λ‹ˆλ‹€.
@rlatjcj pymupdf νŒ¨ν‚€μ§€ μΆ”κ°€ν•˜μ˜€μŠ΅λ‹ˆλ‹€.

Copy link
Contributor

@sungchul2 sungchul2 left a comment

Choose a reason for hiding this comment

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

πŸ–₯️ OS: 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, ν•œκ΅­μ–΄ 기술 κ°€λŠ₯}
    • κ³ μƒλ§ŽμœΌμ…¨μŠ΅λ‹ˆλ‹€!

Copy link
Contributor

@ppakyeah ppakyeah left a comment

Choose a reason for hiding this comment

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

πŸ–₯️ OS:Mac
βœ… 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, ν•œκ΅­μ–΄ 기술 κ°€λŠ₯}
    κ³ μƒν•˜μ…¨μŠ΅λ‹ˆλ‹€ !!

@teddylee777 teddylee777 merged commit c2dbb6e into LangChain-OpenTutorial:main Jan 18, 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
docs tutorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants