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

Skip to content

Conversation

Hye-yoonJeong
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

[Title] Self-querying
[Version] initial
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Self-querying
[Version] initial
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Prompt Template
[Version] revision
[Language] ENG
[Packages] langchain_core.prompts, langchain_openai
[Title] Self-querying
[Version] revision
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Self-querying
[Version] revision
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Self-querying
[Version] revision
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Self-querying
[Version] revision
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Self-querying
[Version] revision
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
[Title] Self-querying
[Version] revision
[Language] ENG
[Packages] langchain.chains.query_constructor,
langchain.retrievers.self_query, langchain_chroma, langchain_openai
@Hye-yoonJeong
Copy link
Contributor Author

1μ£Όμ°¨ κ²€μˆ˜ 파일과 2μ£Όμ°¨ 개발 νŒŒμΌμ„ λΆ„λ¦¬ν•˜μ—¬ PR μƒμ„±ν•˜κΈ° μœ„ν•΄ 기쑴의 PR을 λ‹«κ³  μƒˆλ‘œ PR μ—΄μ—ˆμŠ΅λ‹ˆλ‹€. @harheem @sirena1 파일 리뷰 ν•œ 번만 더 λΆ€νƒλ“œλ¦½λ‹ˆλ‹€. κ°μ‚¬ν•©λ‹ˆλ‹€.

Copy link
Contributor

@harheem harheem 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: νŒ¨ν‚€μ§€ 이슈둜 어렀움이 λ§ŽμœΌμ…¨μ„ 텐데 λκΉŒμ§€ ν•΄κ²°ν•΄ λ‚΄μ‹  것 정말 λŒ€λ‹¨ν•˜μ‹­λ‹ˆλ‹€. νŠœν† λ¦¬μ–Όλ„ λˆ„κ΅¬λ‚˜ μ‰½κ²Œ 이해할 수 μžˆλ„λ‘ μ •μ„±μŠ€λŸ½κ²Œ μž‘μ„±ν•΄ μ£Όμ…¨λ„€μš”! λ‹€λ₯Έ κ°œλ°œμžλΆ„λ“€μ—κ²Œλ„ 쒋은 ν•™μŠ΅ μžλ£Œκ°€ 될 것 κ°™μŠ΅λ‹ˆλ‹€. 저도 λ‹€μŒ μ½˜ν…μΈ  μ€€λΉ„ν•  λ•Œ λ§Žμ€ μ˜κ°μ„ μ–»μ—ˆμ–΄μš”. κ°μ‚¬ν•©λ‹ˆλ‹€ :)

Copy link
Contributor

@sirena1 sirena1 left a comment

Choose a reason for hiding this comment

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

[Review Checklist]

  • Review OS:

    • Windows
  • Template Rule μ€€μˆ˜ μ—¬λΆ€:

    • Template Rule κ°€μ΄λ“œλ₯Ό μ€€μˆ˜ν•˜μ˜€λŠ”κ°€? (Check ν‘œκΈ°)
  • Table of Contents 링크 확인:

    • Table of Contents의 링크가 μ›ν™œν•˜κ²Œ λ™μž‘ν•˜λŠ”μ§€ ν™•μΈν•˜μ˜€λŠ”κ°€? (Check ν‘œκΈ°)
  • 이미지 파일λͺ… κ²€ν† :

    • 이미지가 ν¬ν•¨λ˜μ–΄ μžˆλ‹€λ©΄, μ΄λ―Έμ§€μ˜ 파일λͺ…이 κ°€μ΄λ“œλ₯Ό μ€€μˆ˜ν•˜μ˜€λŠ”κ°€? (Check ν‘œκΈ°)
  • μ΅œμ‹  import 방식 μ‚¬μš© μ—¬λΆ€:

    • import ꡬ문이 μ˜ˆμ „ legacy 방식이 μ•„λ‹Œ μ΅œμ‹  버전을 λ”°λ₯΄λŠ”κ°€? (Check ν‘œκΈ°)
  • μ½”λ“œ λ™μž‘ 확인:

    • λͺ¨λ“  μ½”λ“œκ°€ 였λ₯˜ 없이 λ™μž‘ν•˜λŠ”κ°€? (Check ν‘œκΈ°)
    • Warning λ°œμƒ μ‹œ μ½”λ©˜νŠΈμ— μ μ–΄μ£Όμ„Έμš”.
  • 제좜 파일 확인:

    • κΌ­ ν•„μš”ν•œ νŒŒμΌλ“€λ§Œ μ œμΆœλ˜μ—ˆλŠ”κ°€?
      • νŠœν† λ¦¬μ–Ό μ™Έ μ„€μ • νŒŒμΌμ΄λ‚˜ λ‹€λ₯Έ 뢄이 μž‘μ—…ν•œ 파일이 ν¬ν•¨λ˜μ—ˆλŠ”μ§€ Files Changedμ—μ„œ 확인 λΆ€νƒλ“œλ¦½λ‹ˆλ‹€. (Check ν‘œκΈ°)
  • 기타 의견:
    고생 λ§ŽμœΌμ…¨μŠ΅λ‹ˆλ‹€ :)

@teddylee777 teddylee777 merged commit 7aa3cc2 into LangChain-OpenTutorial:main Jan 14, 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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants