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

Skip to content

Proofread: < 19-Cookbook / 07-Agent / 15-CoT-basedSmartWebSearch > #847

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Apr 26, 2025
Merged

Proofread: < 19-Cookbook / 07-Agent / 15-CoT-basedSmartWebSearch > #847

merged 2 commits into from
Apr 26, 2025

Conversation

syshin0116
Copy link
Contributor

@syshin0116 syshin0116 commented Mar 7, 2025 •
edited
Loading

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

#846 에 대한 수정사항:

  • line 422
    • 변경 내용: 설명 글에서는 plan steps node 보다는 plan_steps_node function 이 어떨지 의견 드립니다.
      => 고민 후 원상태로 두는게 낫다는 의견입니다.
  • line 610
    • 변경 내용: fast_model (or smart_model) → bold 처리 보다는 backtick 처리가 어떨까 싶습니다.
      => 반영했습니다! fast_model -> fast_model, smart_model -> smart_model

고생 많으십니다! 항상 감사드립니다.

@syshin0116
Proofread: [N-1] 07-Agent / 15-CoT-basedSmartWebSearch
d004e59 
- replace **fast_model** -> `fast_model`
- replace **smart_model** -> `smart_model`
@syshin0116 syshin0116 self-assigned this Mar 7, 2025
kkam-dragon
kkam-dragon previously approved these changes Mar 9, 2025
View reviewed changes
Copy link
Contributor

@kkam-dragon kkam-dragon left a comment

Choose a reason for hiding this comment

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

수고많으셨어요~!
혹시나 검수팀 PM님 추가 의견이 있을수도 있으니 참고하시고요~
파이팅 입니닷!

@BokyungisaGod
Copy link
Contributor

@syshin0116 님 안녕하세요, 수정사항이 있습니다!

스크린샷 2025-03-11 오후 6 12 04

Install required packages에 대한 설명은 의도하신건가요?? 설명으로 코드를 작성하는 것보다 설명을 적어놓는 것이 좋을 것 같은데, 어떻게 생각하시는지 궁금합니다!

스크린샷 2025-03-11 오후 6 26 24

TaskStep tracks a single step (either search, think, or respond).
TaskSteps holds a list of such steps.
SearchQuries , KeyInformation , and SearchResultData model how we handle web search queries and their results.
Reference and FinalResponse describe how we finalize and format the output response, including references.

backtick은 변수명, 함수명, 모듈명에만 하면 됩니다. 혹시라도 강조를 하고싶으시다면 bold처리를 하는 것이 좋을 것 같습니다. 확인 후, 1. 코멘트 or 2. 수정 후 커밋 해주세요! ( + bold처리 뒤에는 한칸 space 처리 하는 것 잊지 마세요~! )

스크린샷 2025-03-11 오후 6 33 46

AgentState, WorkerState 도 마찬가지로 클래스명이기 때문에 backtick보다는 bold가 적절해 보입니다.

스크린샷 2025-03-11 오후 6 49 21

StateGraph -> backtick보다는 bold처리가 적절해 보입니다.

스크린샷 2025-03-12 오후 8 16 44

Impact of AI on Global Trade Efficiency During the Pandemic 이란 제목 밑에 1~2줄 설명 추가 부탁드립니다!

스크린샷 2025-03-12 오후 8 19 06

원본 링크는 최대한 보이지 않게 첨부 부탁드립니다.
-> https://hypersonix.ai/blo 그대로 쓰지 않고, source 라고 작성

스크린샷 2025-03-12 오후 8 20 39

Impact of AI on Global Trade Efficiency During the Pandemic라는 제목 아래에 설명 1~2줄 추가 부탁드립니다.

스크린샷 2025-03-12 오후 8 21 19

Impact of AI on Global Trade Efficiency During the Pandemic 라는 제목 아래에 설명 1~2줄 추가 부탁드립니다.

수정 후 커밋 해주시면 되고, 제가 잘못 말씀드린 부분이 있다면 코멘트 남겨주시면 확인해보겠습니다~

@syshin0116
Proofread: [N-1] 07-Agent / 15-CoT-basedSmartWebSearch
f2d649e 
- replace class backticks  to bold
- remove unintended code
@syshin0116
Copy link
Contributor Author

syshin0116 commented Mar 14, 2025 •
edited
Loading

검토 감사드립니다!

Note

아래 각 수정 요청에 대해 Note Callout로 반영 내용 작성했습니다.

스크린샷 2025-03-11 오후 6 12 04

Install required packages에 대한 설명은 의도하신건가요?? 설명으로 코드를 작성하는 것보다 설명을 적어놓는 것이 좋을 것 같은데, 어떻게 생각하시는지 궁금합니다!

Note

의도치 않은 코드였습니다. 제거했습니다. 발견 감사드립니다.

스크린샷 2025-03-11 오후 6 26 24

TaskStep tracks a single step (either search, think, or respond).
TaskSteps holds a list of such steps.
SearchQuries , KeyInformation , and SearchResultData model how we handle web search queries and their results.
Reference and FinalResponse describe how we finalize and format the output response, including references.

backtick은 변수명, 함수명, 모듈명에만 하면 됩니다. 혹시라도 강조를 하고싶으시다면 bold처리를 하는 것이 좋을 것 같습니다. 확인 후, 1. 코멘트 or 2. 수정 후 커밋 해주세요! ( + bold처리 뒤에는 한칸 space 처리 하는 것 잊지 마세요~! )

Note

제안주신대로 변경했습니다.

스크린샷 2025-03-11 오후 6 33 46

AgentState, WorkerState 도 마찬가지로 클래스명이기 때문에 backtick보다는 bold가 적절해 보입니다.

Note

제안주신대로 backtick → bold로 변경했습니다.

스크린샷 2025-03-11 오후 6 49 21

StateGraph -> backtick보다는 bold처리가 적절해 보입니다.

Note

제안주신대로 backtick → bold로 변경했습니다.

스크린샷 2025-03-12 오후 8 16 44

Impact of AI on Global Trade Efficiency During the Pandemic 이란 제목 밑에 1~2줄 설명 추가 부탁드립니다!

Warning

여기부터 아래 내용은 제가 작성한게 아닌 LLM 생성 결과이라 제안주신 내용 반영하지 않았습니다

스크린샷 2025-03-12 오후 8 19 06

원본 링크는 최대한 보이지 않게 첨부 부탁드립니다.
-> https://hypersonix.ai/blo 그대로 쓰지 않고, source 라고 작성

Warning

LLM 생성 결과이라 제안주신 내용 반영하지 않았습니다

스크린샷 2025-03-12 오후 8 20 39

Impact of AI on Global Trade Efficiency During the Pandemic라는 제목 아래에 설명 1~2줄 추가 부탁드립니다.

Warning

LLM 생성 결과이라 제안주신 내용 반영하지 않았습니다

스크린샷 2025-03-12 오후 8 21 19

Impact of AI on Global Trade Efficiency During the Pandemic 라는 제목 아래에 설명 1~2줄 추가 부탁드립니다.

Warning

LLM 생성 결과이라 제안주신 내용 반영하지 않았습니다

검토 감사드립니다!!

@BokyungisaGod
Copy link
Contributor

ㅎㅎㅎ그런거였군요!! 확인했습니다!😊

BokyungisaGod
BokyungisaGod approved these changes Mar 16, 2025
View reviewed changes
Copy link
Contributor

@BokyungisaGod BokyungisaGod left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 :)

@syshin0116 syshin0116 requested a review from kkam-dragon March 19, 2025 01:50
@syshin0116
Copy link
Contributor Author

@kkam-dragon 수정사항이 있어 재aprrove 부탁드립니다!

@teddylee777 teddylee777 merged commit 053baa4 into LangChain-OpenTutorial:main Apr 26, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BokyungisaGod BokyungisaGod BokyungisaGod approved these changes

@kkam-dragon kkam-dragon Awaiting requested review from kkam-dragon

Assignees

@syshin0116 syshin0116

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@syshin0116 @BokyungisaGod @kkam-dragon @teddylee777