[E-4] 13-LCEL / 10-Binding #197

BaBetterB · 2025-01-07T00:58:49Z

[E-4] 13-LCEL / 10-Binding
[Title] Binding
[Version] initial commit
[Language] ENG
[Package] langsmith,langchain,langchain_core,langchain_openai
디스코드에 사전 제공된 openai api 세팅 후 검토 하시면 됩니다.^^

{PR Message - Write freely, remove this bracket if unnecessary}7

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] Binding [Version] initial commit [Language] ENG [Package] langsmith,langchain,langchain_core,langchain_openai

gyjong

[Review Checklist]

Review OS:
- Windows / Mac / Linux (Mac)
Template Rule 준수 여부:
- Template Rule 가이드를 준수하였는가? (Check 표기)
Table of Contents 링크 확인:
- Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
이미지 파일명 검토:
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
최신 import 방식 사용 여부:
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
코드 동작 확인:
- 모든 코드가 오류 없이 동작하는가? (Check 표기)
- Warning 발생 시 코멘트에 적어주세요.
제출 파일 확인:
- 꼭 필요한 파일들만 제출되었는가?
  - 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
기타 의견:
!pip > %pip 그리고 gpt-4 > gpt-4o (또는 gpt-4o-mini)로 수정하면 될것 같습니다.
다른 부분 문제 없이 잘 작동 합니다.

gyjong · 2025-01-07T05:54:44Z

13-LangChain-Expression-Language/10-Binding.ipynb

+   ],
+   "source": [
+    "%%capture --no-stderr\n",
+    "!pip install langchain-opentutorial"


!pip install langchain-opentutorial 대신 %pip install langchain-opentutorial 적용 필요 할것 같습니다.
Discord 링크

Claude에 물어 보면 이렇다고 합니다.
!pip: 시스템 셸에서 실행
%pip: Jupyter 커널 내부에서 실행

상세한 리뷰 감사드립니다^^ 수정해서 commit 하겠습니다.

gyjong · 2025-01-07T06:08:09Z

13-LangChain-Expression-Language/10-Binding.ipynb

+    ")\n",
+    "\n",
+    "\n",
+    "model = ChatOpenAI(model=\"gpt-4\", temperature=0).bind(\n",


model 은 gpt-4o 또는 got-4o-mini 적용하면 명확할것 같습니다. 감사 합니다.
Discord 링크

gyjong · 2025-01-07T06:08:16Z

13-LangChain-Expression-Language/10-Binding.ipynb

+    "    ]\n",
+    ")\n",
+    "# Initialize the ChatOpenAI model and set temperature to 0.\n",
+    "model = ChatOpenAI(model=\"gpt-4\", temperature=0)\n",


model 은 gpt-4o 또는 got-4o-mini 적용하면 명확할것 같습니다. 감사 합니다.
Discord 링크

EQUATION: ( x^3 + 7 = 12 )

SOLUTION:

Subtract 7 from both sides of the equation:
[
x^3 + 7 - 7 = 12 - 7
]
[
x^3 = 5
]

Take the cube root of both sides to solve for ( x ):
[
x = \sqrt[3]{5}
]

Therefore, the solution is:
[
x = \sqrt[3]{5}
]

Gpt-4o의 경우 LaTeX-style 문제가 발생하여 아래 문구를 추가하였습니다.

#"Please avoid LaTeX-style formatting and use plain symbols."

gyjong · 2025-01-07T06:15:11Z

13-LangChain-Expression-Language/10-Binding.ipynb

+   ],
+   "source": [
+    "# Initialize the ChatOpenAI model and bind the tools.\n",
+    "model = ChatOpenAI(model=\"gpt-4\").bind(tools=tools)\n",


model 은 gpt-4o 또는 got-4o-mini 적용하면 명확할것 같습니다. 감사 합니다.
Discord 링크

네 수정완료하였습니다^^

gyjong

[Review Checklist]

Review OS:
- Windows / Mac / Linux (Mac)
Template Rule 준수 여부:
- Template Rule 가이드를 준수하였는가? (Check 표기)
Table of Contents 링크 확인:
- Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
이미지 파일명 검토:
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
최신 import 방식 사용 여부:
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
코드 동작 확인:
- 모든 코드가 오류 없이 동작하는가? (Check 표기)
- Warning 발생 시 코멘트에 적어주세요.
제출 파일 확인:
- 꼭 필요한 파일들만 제출되었는가?
  - 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
기타 의견:
잘 작동 합니다. 수고 하셨습니다.

…guage] ENG [Package] langsmith,langchain,langchain_core,langchain_openai

ppakyeah

[Review Checklist]

Review OS:
- Mac
Template Rule 준수 여부:
- Template Rule 가이드를 준수하였는가? (Check 표기)
Table of Contents 링크 확인:
- Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
이미지 파일명 검토:
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
최신 import 방식 사용 여부:
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
코드 동작 확인:
- 모든 코드가 오류 없이 동작하는가? (Check 표기)
- Warning 발생 시 코멘트에 적어주세요.
제출 파일 확인:
- 꼭 필요한 파일들만 제출되었는가?
  - 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
기타 의견:
추가적인 의견이나 수정 사항을 기재해주세요.
정상 동작합니다 고생하셨습니다! 사소한 의견 하나 있는데 필수 수정 사항은 아닙니다!

ppakyeah · 2025-01-08T03:32:49Z

13-LangChain-Expression-Language/10-Binding.ipynb

+    "            # Write the following equation using algebraic symbols and then solve it.\n",
+    "            \"Write out the following equation using algebraic symbols then solve it. \"\n",


사소하게 해당 주석은 한국어 튜토리얼에서 번역을 위해 사용되었던 걸로 보여서 제거되어도 괜찮지 않을까 의견드립니다!

섬세한 리뷰 감사드립니다^^

gyjong

[Review Checklist]

Review OS:
- Windows / Mac / Linux (Mac)
Template Rule 준수 여부:
- Template Rule 가이드를 준수하였는가? (Check 표기)
Table of Contents 링크 확인:
- Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
이미지 파일명 검토:
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
최신 import 방식 사용 여부:
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
코드 동작 확인:
- 모든 코드가 오류 없이 동작하는가? (Check 표기)
- Warning 발생 시 코멘트에 적어주세요.
제출 파일 확인:
- 꼭 필요한 파일들만 제출되었는가?
  - 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
기타 의견:
특이사항 없습니다. 내용 확인 및 코드 정상 작동 모두 확인 하였습니다.
수고 많으셨습니다.

BaBetterB added 3 commits January 7, 2025 00:24

[E-4] 13-LCEL / 10-Binding [Title] Binding

3962589

Merge remote-tracking branch 'upstream/main'

d879139

[E-4] 13-LCEL / 10-Binding

01a2b7f

[Title] Binding [Version] initial commit [Language] ENG [Package] langsmith,langchain,langchain_core,langchain_openai

BaBetterB requested review from gyjong and ppakyeah January 7, 2025 00:58

BaBetterB self-assigned this Jan 7, 2025

gyjong approved these changes Jan 7, 2025

View reviewed changes

gyjong previously approved these changes Jan 7, 2025

View reviewed changes

BaBetterB added 2 commits January 7, 2025 23:31

Merge branch 'LangChain-OpenTutorial:main' into main

b72347d

[E-4] 13-LCEL / 10-Binding [Title] Binding [Version] fixed issue [Lan…

d70ad63

…guage] ENG [Package] langsmith,langchain,langchain_core,langchain_openai

BaBetterB dismissed gyjong’s stale review via d70ad63 January 7, 2025 16:04

ppakyeah approved these changes Jan 8, 2025

View reviewed changes

gyjong approved these changes Jan 8, 2025

View reviewed changes

teddylee777 merged commit b64a602 into LangChain-OpenTutorial:main Jan 9, 2025
1 of 2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[E-4] 13-LCEL / 10-Binding #197

[E-4] 13-LCEL / 10-Binding #197

Uh oh!

BaBetterB commented Jan 7, 2025

Uh oh!

gyjong left a comment •

edited by JunseongKim1104

Loading

Uh oh!

gyjong Jan 7, 2025

Uh oh!

BaBetterB Jan 7, 2025

Uh oh!

gyjong Jan 7, 2025

Uh oh!

gyjong Jan 7, 2025

Uh oh!

BaBetterB Jan 7, 2025

Uh oh!

gyjong Jan 7, 2025

Uh oh!

BaBetterB Jan 7, 2025

Uh oh!

gyjong left a comment

Uh oh!

ppakyeah left a comment •

edited

Loading

Uh oh!

ppakyeah Jan 8, 2025

Uh oh!

BaBetterB Jan 9, 2025

Uh oh!

gyjong left a comment

Uh oh!

Uh oh!

Uh oh!

		" # Write the following equation using algebraic symbols and then solve it.\n",
		" \"Write out the following equation using algebraic symbols then solve it. \"\n",

[E-4] 13-LCEL / 10-Binding #197

[E-4] 13-LCEL / 10-Binding #197

Uh oh!

Conversation

BaBetterB commented Jan 7, 2025

Author Checklist

Review Template (Intial PR)

Uh oh!

gyjong left a comment • edited by JunseongKim1104 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

[Review Checklist]

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gyjong left a comment

Choose a reason for hiding this comment

[Review Checklist]

Uh oh!

ppakyeah left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

[Review Checklist]

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gyjong left a comment

Choose a reason for hiding this comment

[Review Checklist]

Uh oh!

Uh oh!

Uh oh!

gyjong left a comment •

edited by JunseongKim1104

Loading

ppakyeah left a comment •

edited

Loading