[E-3] 17-LangGraph/02-Structures-LangGraph-Add-Groundedness-Check.ipynb #471

jeongkpa · 2025-01-20T16:17:02Z

{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

jeongkpa · 2025-01-22T14:56:52Z

@changwonjeon @LEE1026icarus 님 PR 남깁니다. 리뷰 잘 부탁드려요!!

LEE1026icarus

🖥️ 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.
Comments: {Write freely, 한국어 기술 가능}

17-LangGraph/02-Structures/03-LangGraph-Add-Groundedness-Check.ipynb

samdaseuss

🖥️ 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, 한국어 기술 가능}
수고하셨습니다 :)

그런데 rag 폴더의 파일은 충돌 때문에 올리지 않는걸로 알고 있는데 혹시 모르니 일단 보류해놓겠습니다!

changwonjeon · 2025-01-24T06:29:25Z

@changwonjeon @LEE1026icarus 님 PR 남깁니다. 리뷰 잘 부탁드려요!!

코멘트만 저를 언급해주시고 PR 요청이 없어서, 인제 확인했네요.
제가 하면 되나요? reivew request 주세요!

아래 그림처럼, request 받지 않아서 안보입니다.
필요하시면 reivew request 남겨주세요.

jeongkpa · 2025-01-25T03:09:44Z

🖥️ 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, 한국어 기술 가능}
수고하셨습니다 :)

그런데 rag 폴더의 파일은 충돌 때문에 올리지 않는걸로 알고 있는데 혹시 모르니 일단 보류해놓겠습니다!

🥕배기민 (Infra) — 2025-01-23 오후 6:04
@everyone 현재 langchain-teddynote에 있는 기능 중 document_loaders, tools, evaluator, graphs, messages는 langchain-opentutorial 로 이식했습니다.

from langchain_teddynote 에서 from langchain_opentutorial 로 변경하시면 바로 이용가능합니다! 혹시나 오류 나는 경우 바로 ⁠인프라-개발 과 깃헙 이슈로 남겨주세요!

이부분 확인했고, RAG 폴더 삭제 후 수정하겠습니다!

changwonjeon

[Review Checklist]

Review OS: Windows
Template Rule 준수 여부: 이상없음
Table of Contents 링크 확인: 연결은 이상없으나, 소문자 수정 필요
- 제안 드린대로, (# )안에 내용은 소문자로 권장되고 있으니 확인 부탁드립니다.
이미지 파일명 검토: 이상없음
최신 import 방식 사용 여부: 이상없음
- 다만, 필요없는 import 가 있는 듯 하여, 확인 부탁드립니다.
코드 동작 확인: 이상없음
제출 파일 확인: data 폴더의 pdf파일 이상없음
기타 의견:
코드는 이상/Warning 없이 잘 실행되었습니다
작성하시느라 고생 많으셨습니다.
다만, TOC의 작성 가이드를 한번 확인해주시면 좋을 듯 합니다. (# )안에 소문자 사용관련 확인이 필요할 듯 하네요.
또한, langchain-community가 실제로 사용되는지도 확인 부탁드립니다.

changwonjeon · 2025-01-25T10:34:59Z

17-LangGraph/02-Structures/03-LangGraph-Add-Groundedness-Check.ipynb

+    "- [Basic PDF-based Retrieval Chain](#Basic-PDF-based-Retrieval-Chain)\n",
+    "- [Defining State](#Defining-State)\n",
+    "- [Degining Nodes](#Degining-Nodes)\n",
+    "- [Defining Edges](#Defining-Edges)\n",
+    "- [Running a graph](#Running-a-graph)\n",


Suggested change

"- [Basic PDF-based Retrieval Chain](#Basic-PDF-based-Retrieval-Chain)\n",

"- [Defining State](#Defining-State)\n",

"- [Degining Nodes](#Degining-Nodes)\n",

"- [Defining Edges](#Defining-Edges)\n",

"- [Running a graph](#Running-a-graph)\n",

"- [Basic PDF-based Retrieval Chain](#basic-pdf-based-retrieval-chain)\n",

"- [Defining State](#defining-state)\n",

"- [Degining Nodes](#degining-nodes)\n",

"- [Defining Edges](#defining-edges)\n",

"- [Running a graph](#running-a-graph)\n",

(영문) https://github.com/LangChain-OpenTutorial/LangChain-OpenTutorial/blob/main/99-TEMPLATE/English/00-BASE-TEMPLATE-GUIDE.ipynb

(한글) https://github.com/LangChain-OpenTutorial/LangChain-OpenTutorial/blob/main/99-TEMPLATE/Korean/00-BASE-TEMPLATE-GUIDE.ipynb
=>
Anchor 관련 가이드
목차에 들어가는 anchor는 각 헤딩에 정확히 일치하는 text를 #기호 다음 모두 소문자로, 대시는 하나만 사용
예시: #environment-setup
잘못된 예시: #Environment-Setup, #environment--setup
참고: 깃헙 docs에서 Section links

에 따르면, ( )안의 #뒤에 내용은 모두 소문자를 권장합니다.
바로 윗줄 Environment Setup은 ()안에는 소문자로 되어있는 듯 한데,
그 하단부터는 대문자가 있어서 수정을 권장드립니다.
확인 부탁드립니다.

changwonjeon · 2025-01-25T10:36:29Z

17-LangGraph/02-Structures/03-LangGraph-Add-Groundedness-Check.ipynb

+    "        \"langchain\",\n",
+    "        \"python-dotenv\",\n",
+    "        \"langchain-core\",     \n",
+    "        \"langchain-community\",\n",


Suggested change

" \"langchain-community\",\n",

해당 패키지는 사용되지 않는 듯 합니다.
주석처리하고 실행하니, 결과 나오는데 지장이 없었습니다.
확인 부탁드립니다.

jeongkpa · 2025-01-25T13:35:04Z

@samdaseuss 오윤지님 상세한 피드백 주셔서 대단히 감사드립니다! 모두 수정하였습니다!

jeongkpa · 2025-01-25T13:35:40Z

@LEE1026icarus 순형님 모두 수정했습니다!

LEE1026icarus

유익한 콘텐츠 제작하시느라 고생하셨습니다 ㅎㅎ

samdaseuss · 2025-02-02T23:44:05Z

🖥️ 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, 한국어 기술 가능}
수고하셨습니다 :)

chaeyoonyunakim

번역/검수 확인하였습니다

changwonjeon

[Review Checklist]

Review OS: Windows
Template Rule 준수 여부: 이상없음
Table of Contents 링크 확인: 수정 후 이상없음
이미지 파일명 검토: 이상없음
최신 import 방식 사용 여부: 이상없음
코드 동작 확인: 이상없음
제출 파일 확인: data 폴더의 pdf파일 이상없음
기타 의견:
알람이 없어서, 지나간 줄 알았습니다. 늦은 확인 양해부탁드립니다.
이전에도 코드 실행은 이상없었고, 지금 현재 상태로도 이상없이 잘 작동합니다.
approve 으로 바로 하긴 하는데, 코드 중간에 스타일이 약간 다른 부분이 있어서 제안드렸으니 확인 부탁드립니다.
고생많으셨습니다. 감사합니다.

changwonjeon · 2025-02-04T10:45:28Z

17-LangGraph/02-Structures/03-LangGraph-Add-Groundedness-Check.ipynb

+    "        llm=ChatOpenAI(model=\"gpt-4o-mini\", temperature=0), \n",
+    "        target=\"question-retrieval\"\n",


Suggested change

" llm=ChatOpenAI(model=\"gpt-4o-mini\", temperature=0), \n",

" target=\"question-retrieval\"\n",

" llm = ChatOpenAI(model=\"gpt-4o-mini\", temperature=0), \n",

" target = \"question-retrieval\"\n",

코드는 이상없이 잘 실행됩니다.
다만, 코딩 스타일이 '='앞뒤로 공백을 한 칸씩 두는데,
이 부분만 붙어있습니다.
수정 제안 드립니다.

[E-3] 17-LangGraph/02-Structures-LangGraph-Add-Groundedness-Check.ipynb

c8f6502

jeongkpa self-assigned this Jan 20, 2025

chaeyoonyunakim self-requested a review January 21, 2025 14:08

chaeyoonyunakim added proofreading 번역/검수팀 제안사항 반영 docs tutorial labels Jan 21, 2025

fix : typo

f89e72e

jeongkpa requested review from samdaseuss and LEE1026icarus January 22, 2025 14:33

LEE1026icarus reviewed Jan 23, 2025

View reviewed changes

17-LangGraph/02-Structures/03-LangGraph-Add-Groundedness-Check.ipynb Outdated Show resolved Hide resolved

samdaseuss requested changes Jan 23, 2025

View reviewed changes

teddylee777 requested a review from changwonjeon January 24, 2025 14:28

fix : 빈 셀 제거

be0053b

changwonjeon reviewed Jan 25, 2025

View reviewed changes

jeongkpa added 2 commits January 25, 2025 22:32

fix : rag 폴더 삭제

b756829

fix : 소문자 변경

95e09af

jeongkpa requested a review from samdaseuss January 25, 2025 13:34

LEE1026icarus self-requested a review January 25, 2025 13:36

LEE1026icarus approved these changes Jan 25, 2025

View reviewed changes

samdaseuss approved these changes Feb 2, 2025

View reviewed changes

chaeyoonyunakim approved these changes Feb 3, 2025

View reviewed changes

changwonjeon approved these changes Feb 4, 2025

View reviewed changes

teddylee777 merged commit 9d58c3f into LangChain-OpenTutorial:main Feb 8, 2025
3 of 4 checks passed

		" llm=ChatOpenAI(model=\"gpt-4o-mini\", temperature=0), \n",
		" target=\"question-retrieval\"\n",

[E-3] 17-LangGraph/02-Structures-LangGraph-Add-Groundedness-Check.ipynb #471

[E-3] 17-LangGraph/02-Structures-LangGraph-Add-Groundedness-Check.ipynb #471

Uh oh!

Conversation

jeongkpa commented Jan 20, 2025

Author Checklist

Review Template (Intial PR)

Uh oh!

jeongkpa commented Jan 22, 2025

Uh oh!

LEE1026icarus 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

Uh oh!

Uh oh!

samdaseuss left a comment

Choose a reason for hiding this comment

Uh oh!

changwonjeon commented Jan 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jeongkpa commented Jan 25, 2025

Uh oh!

changwonjeon 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!

changwonjeon Jan 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

changwonjeon Jan 25, 2025

Choose a reason for hiding this comment

Uh oh!

jeongkpa commented Jan 25, 2025

Uh oh!

jeongkpa commented Jan 25, 2025

Uh oh!

LEE1026icarus left a comment

Choose a reason for hiding this comment

Uh oh!

samdaseuss commented Feb 2, 2025

Uh oh!

chaeyoonyunakim left a comment

Choose a reason for hiding this comment

Uh oh!

changwonjeon left a comment

Choose a reason for hiding this comment

[Review Checklist]

Uh oh!

changwonjeon Feb 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

LEE1026icarus left a comment •

edited

Loading

changwonjeon commented Jan 24, 2025 •

edited

Loading

changwonjeon left a comment •

edited

Loading

changwonjeon Jan 25, 2025 •

edited

Loading