Skip to content

[E-2]03-OutputParser/03-StructedOutputParser#95

Merged
teddylee777 merged 8 commits intoLangChain-OpenTutorial:mainfrom
hohosznta:master
Jan 3, 2025
Merged

[E-2]03-OutputParser/03-StructedOutputParser#95
teddylee777 merged 8 commits intoLangChain-OpenTutorial:mainfrom
hohosznta:master

Conversation

@hohosznta
Copy link
Copy Markdown
Contributor

@hohosznta hohosznta commented Jan 3, 2025

reference 링크 최신화하였고 예시를 글로벌하게 바꾸었습니다.

Copy link
Copy Markdown
Contributor

@ranian963 ranian963 left a comment

Choose a reason for hiding this comment

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

References
LangChain Structured output parser

이 링크가 다른 링크로 되어 있는 것 같습니다.
들어가니 내용은 비슷하나 다른 함수를 호출 하고 있습니다.
PydanticOutputParser로 처리하는 방법을 설명하고 있습니다.

https://python.langchain.com/api_reference/langchain/output_parsers/langchain.output_parsers.structured.StructuredOutputParser.html#langchain.output_parsers.structured.StructuredOutputParser

이 링크가 더 맞을 것 같습니다.
확인 부탁드립니다. 그외에는 다 괜찮은 것 같습니다. 수고하셨습니다!

@hohosznta
Copy link
Copy Markdown
Contributor Author

상윤님 리뷰 감사드립니다. 링크 고치고 커밋하였으니 approve 부탁드립니다!

Copy link
Copy Markdown
Contributor

@asummerz asummerz left a comment

Choose a reason for hiding this comment

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

[Review]

  1. Review OS(Windows/Mac/Linux): Mac
  2. Template Rule 가이드를 준수 하였는가(YES/NO): YES
  3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
  4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): YES
  5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
  6. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES

모두 확인하였습니다. 수고 많으셨습니다!

@asummerz asummerz self-requested a review January 3, 2025 13:15
@teddylee777 teddylee777 merged commit ff32a00 into LangChain-OpenTutorial:main Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants