[E-1] 20-Prompt / Agent prompt #604

Secludor · 2025-02-05T14:54:20Z

{PR Message - Write freely, remove this bracket if unnecessary}
1차 작성 완료했습니다.
에이전트 프롬프트의 개념과 요소에 대한 설명이 포함되어 있고, 에이전트 프롬프트 예제를 좀 더 추가할 예정입니다.
개선사항이나 있으면 좋을 것 같은 예제 제안주시면 적극 반영하겠습니다!

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

Hye-yoonJeong · 2025-02-06T10:45:06Z

안녕하세요 주영님! :) 저는 CoT, ToT, GoT 프롬프트를 담당하고 있는데요, CoT를 적용할 만한 것으로는 아무래도 멀티에이전트의 supervisor가 가장 적절한 것 같아서요. 현재는 아직 머지되진 않았지만 #490 계층적 멀티에이전트를 가지고 실험중이에요. 여기에 supervisor가 세 개가 있는데 프롬프트는 다 같은데 members만 변수로 받아서 만들면 될 것 같아서요. 그런데 다른 단계들이 있다보니 프롬프트가 제대로 작동하는지 확인하기 어려워서 고민중입니다 :) 혹시 모르니까 제가 올린 파일에도 리뷰어 지정해두겠습니다! :)

Secludor · 2025-02-10T02:53:57Z

1차 작성 완료했습니다.
에이전트 프롬프트의 개념과 요소에 대한 설명이 포함되어 있고, 에이전트 프롬프트 예제를 좀 더 추가할 예정입니다.
개선사항이나 있으면 좋을 것 같은 예제 제안주시면 적극 반영하겠습니다!

jinucho

코드 실행 보단,

내용 위주로 검토했습니다.
각 목적 별 프롬프트에 대한 설명이 자세하여 좋습니다.
추가로 프롬프트 사용에 대한 출력 예시도 있어 이해하기에 적합 합니다.

고생하셨습니다.

jinucho

Approve가 해제된것으로 보여 다시 리뷰 제출 합니다!

[amend] diagram,PROMPT_QUALITY_EVALUATION_TEMPLATE

jinucho

추가 및 변경된 내용 확인하였습니다.

Secludor · 2025-02-13T11:33:10Z

프롬프트 사용 예제와 가이드라인 전문을 포함한 ipynb를 만들었는데 이로 대체하거나 사용예제만을 포함해 추가로 커밋하면 어떨까요?

jinucho · 2025-02-14T00:02:33Z

프롬프트 사용 예제와 가이드라인 전문을 포함한 ipynb를 만들었는데 이로 대체하거나 사용예제만을 포함해 추가로 커밋하면 어떨까요?

다른 튜토리얼에 없는 전용 프롬프트라면 예제파일 너무 좋을 것 같습니다~!!
올려 주시면 확인 하겠습니다.

Secludor · 2025-02-14T02:35:41Z

다른 튜토리얼에 없는 전용 프롬프트라면 예제파일 너무 좋을 것 같습니다~!! 올려 주시면 확인 하겠습니다.

감사합니다 :)
md 파일 마지막에 있는 AI_CAREER_PLANNER_AGENT_PROMPT , PROMPT_QUALITY_EVALUATION_TEMPLATE 이 실제 어떻게 사용될 수 있는 지에 대한 예제입니다! 내용 자체는 md 파일과 크게 다르지 않아

둘 중 하나만 올리기
둘 다 올리기
기존 md 파일에 더해 실제 사용 예제만 ipynb에 유지하기
로 생각중입니다!

jinucho

🖥️ 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: 코드 동작 이상 없습니다.

개인적인 추가 의견으로는

md에 있는 내용이 ipynb 내에도 동일하게 있는데, md 파일 내용은 지우고
md 파일내에 ipynb를 참조 링크로 붙이는게 어떨까 싶습니다.

또한 ipynb에서 Table of Contents에 추가하는 항목은 ## 인 경우로 기억합니다.

전용 예제 파일을 만드신거니 내용을 적절히 분리하여 올리시는게 어떨 까요?

환경변수 셋팅에서

from dotenv import load_dotenv
from langchain_opentutorial import set_env

# Attempt to load environment variables from a .env file; if unsuccessful, set them manually.
if not load_dotenv():
    set_env(
        {
            "OPENAI_API_KEY": "",
            "LANGCHAIN_API_KEY": "",
            "LANGCHAIN_TRACING_V2": "true",
            "LANGCHAIN_ENDPOINT": "https://api.smith.langchain.com",
            "TAVILY_API_KEY": "",
        }
    )

# set the project name same as the title
set_env(
    {
        "LANGCHAIN_PROJECT": "agentic-prompt",
    }
)

이 부분을

from dotenv import load_dotenv
from langchain_opentutorial import set_env

# Attempt to load environment variables from a .env file; if unsuccessful, set them manually.
if not load_dotenv():
    set_env(
        {
            "OPENAI_API_KEY": "",
            "LANGCHAIN_API_KEY": "",
            "LANGCHAIN_TRACING_V2": "true",
            "LANGCHAIN_ENDPOINT": "https://api.smith.langchain.com",
            "TAVILY_API_KEY": "",
             "LANGCHAIN_PROJECT": "agentic-prompt",
        }
    )

이렇게 하나로 합쳐도 될 것 같습니다.

Secludor · 2025-02-14T14:56:24Z

@jinucho

아, 좋은 것 같습니다. md 파일의 내용을 유지하고, ipynb에 코드 실행 예제 부분 만을 남겨 md 파일 상단이나 하단에 해당 파일을 참조 링크로 붙이는 것으로 이해하고 작업했습니다.
- md, ipynb 용도 구분
- ipynb 파일 위치를 asset/ 으로 이동 및 md 파일에 참조링크 추가
환경변수 셋팅에서 "LANGCHAIN_PROJECT": "agentic-prompt"를 따로 둔 이유는 if not load_dotenv(): 에 해당 내용을 넣게 되면 .env에서 LANGCHAIN_PROJECT 변수를 편집해줘야 해당 변수 내용이 반영된다는 번거로움이 있을 것으로 생각해 해당 변수 설정을 분리했습니다. (.env가 실행되면 해당 조건문 블럭이 실행되지 않으므로) 혹 제가 잘못 알고 있는 걸까요??

jinucho

🖥️ 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: 환경변수 관리를 통합해야 할 것 같아서 말씀드린것인데, 불편함을 해소하기 위한 것이라면 상관 없을 것 같습니다!

고생하셨습니다 ㅎㅎ

cjLee-cmd · 2025-02-18T00:21:48Z

🖥️ OS: Mac
✅ Checklist

[v] Template: Tutorials follows the required template.
[v] Table of Contents(TOC) Links: All Table of Contents links work. (Yes/No)
[v] Image: Image filenames follow guidelines.
[v] Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
[v] Code Execution: Code runs without errors.
Comments: {Write freely, 한국어 기술 가능}
잘 만드셨네요. 고생하셨습니다. 다른 리뷰어분 내용 보고 같이 따라해 보았습니다. 정확한 답을 몰라서...
고생하셨습니다~

cjLee-cmd

[승인]

jinucho

@cjLee-cmd 어떤거 잘 못 누르셨나요?

이 PR에 다시 리뷰 요청을 누르셔서 승인 풀렸네요 ㅠㅠ

cjLee-cmd · 2025-02-20T09:23:45Z

@cjLee-cmd 어떤거 잘 못 누르셨나요?

이 PR에 다시 리뷰 요청을 누르셔서 승인 풀렸네요 ㅠㅠ

앗... 제가 뭘 잘못눌렀을까요...?? 승인 요청와서 승인한거 같은데요...
뭘 하면 될까요...--"

jinucho · 2025-02-20T09:27:20Z

@cjLee-cmd 어떤거 잘 못 누르셨나요?
이 PR에 다시 리뷰 요청을 누르셔서 승인 풀렸네요 ㅠㅠ

앗... 제가 뭘 잘못눌렀을까요...?? 승인 요청와서 승인한거 같은데요... 뭘 하면 될까요...--"

다시 한 번 승인만 해주시면 됩니다

cjLee-cmd

[승인] 승인하겠습니다.

[E-1] 20-Prompt / Agent prompt (outline)

5c16415

Secludor and others added 2 commits February 10, 2025 00:49

Merge branch 'LangChain-OpenTutorial:main' into main

f64b2d6

prompt guidlelines with examples

9d345ed

Secludor requested review from cjLee-cmd and jinucho February 10, 2025 02:55

jinucho approved these changes Feb 12, 2025

View reviewed changes

cjLee-cmd requested a review from jinucho February 12, 2025 01:44

jinucho previously approved these changes Feb 12, 2025

View reviewed changes

cjLee-cmd previously approved these changes Feb 12, 2025

View reviewed changes

[add] AI_CAREER_PLANNER_AGENT_PROMPT,

ef0ec16

[amend] diagram,PROMPT_QUALITY_EVALUATION_TEMPLATE

Secludor dismissed stale reviews from cjLee-cmd and jinucho via ef0ec16 February 12, 2025 14:25

file : demo code

f6f0731

jinucho previously approved these changes Feb 13, 2025

View reviewed changes

Secludor requested a review from cjLee-cmd February 13, 2025 07:25

docs : split each of sections to different cells

b93accd

Secludor dismissed jinucho’s stale review via b93accd February 14, 2025 02:27

Secludor requested a review from jinucho February 14, 2025 02:35

jinucho reviewed Feb 14, 2025

View reviewed changes

docs : md, ipynb 용도 구분, ipynb 파일 경로 변경, md파일에 ipynb 참조 링크 추가

6d4ab66

Secludor requested a review from jinucho February 14, 2025 14:56

Merge branch 'LangChain-OpenTutorial:main' into main

f01a190

jinucho approved these changes Feb 16, 2025

View reviewed changes

cjLee-cmd approved these changes Feb 18, 2025

View reviewed changes

cjLee-cmd requested review from jinucho and cjLee-cmd February 18, 2025 05:33

jinucho approved these changes Feb 20, 2025

View reviewed changes

cjLee-cmd approved these changes Feb 20, 2025

View reviewed changes

teddylee777 merged commit cc9735f into LangChain-OpenTutorial:main Feb 21, 2025
1 of 2 checks passed

[E-1] 20-Prompt / Agent prompt #604

[E-1] 20-Prompt / Agent prompt #604

Uh oh!

Conversation

Secludor commented Feb 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Author Checklist

Review Template (Intial PR)

Uh oh!

Hye-yoonJeong commented Feb 6, 2025

Uh oh!

Secludor commented Feb 10, 2025

Uh oh!

jinucho left a comment

Choose a reason for hiding this comment

Uh oh!

jinucho left a comment

Choose a reason for hiding this comment

Uh oh!

jinucho left a comment

Choose a reason for hiding this comment

Uh oh!

Secludor commented Feb 13, 2025

Uh oh!

jinucho commented Feb 14, 2025

Uh oh!

Secludor commented Feb 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

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

Secludor commented Feb 14, 2025

Uh oh!

jinucho left a comment

Choose a reason for hiding this comment

Uh oh!

cjLee-cmd commented Feb 18, 2025

Uh oh!

cjLee-cmd left a comment

Choose a reason for hiding this comment

Uh oh!

jinucho left a comment

Choose a reason for hiding this comment

Uh oh!

cjLee-cmd commented Feb 20, 2025

Uh oh!

jinucho commented Feb 20, 2025

Uh oh!

cjLee-cmd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Secludor commented Feb 5, 2025 •

edited

Loading

Secludor commented Feb 14, 2025 •

edited

Loading

jinucho left a comment •

edited

Loading