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

Skip to content

Conversation

stsr1284
Copy link
Contributor

@stsr1284 stsr1284 commented Jan 13, 2025

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

Author Checklist

  • [ O ] PR Title Format: I have confirmed that the PR title follows the correct format. (e.g., [N-2] 07-Text Splitter / 07-RecursiveCharacterTextSplitter)

  • [ O ] 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.

  1. 첫 서두의 설명에서 일부 구현체에 대해서 백틱 처리가 빠져있습니다.
This tutorial demonstrates how to use the DatetimeOutputParser to:

Set up and initialize the parser for `datetime` generation
Convert a `datetime` object to a string

-> 수정하였습니다!

  1. 18번 셀에서 의존성 설치 명령어는 아래와 같은 포멧으로 작성되어야 합니다.
%%capture --no-stderr
%pip install langchain-opentutorial

-> 수정하였습니다!

  1. 마이너한 문법적 이슈인데, 이 부분은 소유격으로 풀어내는 것이 조금 더 자연스럽습니다.

origin text

If you need to generate output in the form of a date or time, LangChain's `DatetimeOutputParser` simplifies the process.
The `format` of the `DatetimeOutputParser` can be specified by referring to the table below.

recommandation

If you need to generate output in the form of a date or time, the `DatetimeOutputParser` from LangChain simplifies the process.
The `format` of the `DatetimeOutputParser` can be specified by referring to the table below.

-> 수정하였습니다!

++ 가능하다면 동기상태 뿐만 아니라 astream과 같은 비동기 상태에서의 DatetimeOutputParser 사용에 대한 예시도 보강되었으면 더 나은 튜토리얼이 될 수 있을 것 같습니다.

-> 추가하였는데 확인 부탁드립니다!

@stsr1284 stsr1284 added the proofreading 번역/검수팀 제안사항 반영 label Jan 13, 2025
@stsr1284 stsr1284 self-assigned this Jan 13, 2025
BokyungisaGod
BokyungisaGod previously approved these changes Jan 14, 2025
@Two-Jay
Copy link

Two-Jay commented Jan 17, 2025

@stsr1284 안녕하세요.
늦게 확인해서 죄송합니다.

다른 모든 구현은 괜찮은데 늦게 발견한 번역 오류가 하나 있습니다. 소제목 중 Implementing the Datetime Output Parser 만 번역을 수정해서 커밋만 올려주실 수 있으실까요? Implementing (Implementation)은 IT 씬에서 '밑바닥부터 구현체를 만들고 구현한다는 의미가 내포가 되어 있어서 Using the Datatime Output Parser` 와 같은 식으로 전달 부탁드립니다.

다른 부분은 틀린 것이 없으니, 이 부분만 수정 부탁드릴게요. 수정확인 되면 바로 approval 올라갑니다.

Implementing the Datetime Output Parser -> Using the Datetime Output Parser
@stsr1284
Copy link
Contributor Author

@Two-Jay
피드백 감사합니다. 해당 내용 수정 완료했습니다~!

@Two-Jay Two-Jay requested a review from BokyungisaGod January 17, 2025 16:18
Copy link

@Two-Jay Two-Jay left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! :)

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.

수고하셨습니다~

@teddylee777 teddylee777 merged commit a176b72 into LangChain-OpenTutorial:main Jan 18, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants