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

Skip to content

Conversation

hj0302
Copy link
Contributor

@hj0302 hj0302 commented Jan 1, 2025

[Review]

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

hj added 2 commits January 1, 2025 14:15
[Title] Google Generative AI
[Version] initial
[Language] ENG
[Package] langsmith, langchain, langchin_core, langchain_google_genai

Langchain and Google Generative AI Tutorial
- A tutorial on using Langchain's  class to access various generative models from Google AI.
[Title] Google Generative AI
[Version] initial
[Language] ENG
[Package] langsmith, langchain, langchin_core, langchain_google_genai

Langchain and Google Generative AI Tutorial
- A tutorial on using Langchain's  class to access various generative models from Google AI.
@hj0302 hj0302 requested review from ivybae and effort-type January 1, 2025 05:50
@hj0302 hj0302 changed the title 코드 리뷰 부탁드립니다. 04-MODEL / 05-GoogleGenerativeAI Jan 1, 2025
Copy link
Contributor

@ivybae ivybae left a comment

Choose a reason for hiding this comment

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

안녕하세요, @hj0302 현종님! 배인진입니다.
고생많으셨습니다.

[Review]

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

hj added 3 commits January 1, 2025 22:58
[Title] Google Generative AI
[Version] revision
[Language] ENG
[Package] langsmith, langchain, langchin_core, langchain_google_genai

Langchain and Google Generative AI Tutorial
- A tutorial on using Langchain's  class to access various generative models from Google AI.
[Title] Google Generative AI
[Version] revision
[Language] ENG
[Package] langsmith, langchain, langchin_core, langchain_google_genai

Langchain and Google Generative AI Tutorial
- A tutorial on using Langchain's  class to access various generative models from Google AI.
Copy link
Member

@effort-type effort-type left a comment

Choose a reason for hiding this comment

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

Review OS(Windows/Mac/Linux): Windows
Template Rule 가이드를 준수 하였는가(YES/NO): Yes
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): Yes
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): 이미지 없음
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): Yes
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요):
아래에 있는 내용은 수정하는 것이 좋을 것 같다 판단되어 의견드립니다!

@ivybae
Copy link
Contributor

ivybae commented Jan 2, 2025

@hj0302 현종님, 마지막 커밋에서 ee5c764 파일을 삭제하고 push 하신 것으로 보여서요!
한번 작업이력 확인 부탁드릴게요 :)

@hj0302
Copy link
Contributor Author

hj0302 commented Jan 2, 2025

@hj0302 현종님, 마지막 커밋에서 ee5c764 파일을 삭제하고 push 하신 것으로 보여서요! 한번 작업이력 확인 부탁드릴게요 :)

@ivybae ee5c764은 사용되지 않는 파일이여서 삭제하였습니다.

@ivybae
Copy link
Contributor

ivybae commented Jan 2, 2025

@hj0302 04-MODEL/05-Google-Generative-AI.ipynb 삭제하신 파일은 작업하셨던 파일로 보여서요!

@hj0302 hj0302 self-assigned this Jan 2, 2025
ivybae
ivybae previously approved these changes Jan 2, 2025
Copy link
Contributor

@ivybae ivybae left a comment

Choose a reason for hiding this comment

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

@hj0302 force push를 하시면서 커밋 이력 마지막에 작업중이던 파일이 삭제된 것으로 나와 확인을 부탁드렸던 것인데요! 아마도 그 작업 중에 파일 2개로 작업하셨던 거 같네요~
새로운 파일에 요청드렸던 수정사항 모두 반영된 부분 확인하고 Approve 상태로 변경합니다 👍

@hj0302
Copy link
Contributor Author

hj0302 commented Jan 2, 2025

Review OS(Windows/Mac/Linux): Windows Template Rule 가이드를 준수 하였는가(YES/NO): Yes Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): Yes 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): 이미지 없음 import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): Yes 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): 아래에 있는 내용은 수정하는 것이 좋을 것 같다 판단되어 의견드립니다!

@effort-type 안녕하세요!
수정해야 될 내용이 어느 부분인지 확인할 수 있을까요?

Comment on lines 100 to 108
"set_env(\n",
" {\n",
" \"OPENAI_API_KEY\": \"\",\n",
" \"LANGCHAIN_API_KEY\": \"\",\n",
" \"LANGCHAIN_TRACING_V2\": \"true\",\n",
" \"LANGCHAIN_ENDPOINT\": \"https://api.smith.langchain.com\",\n",
" \"LANGCHAIN_PROJECT\": \"05-GoogleGenerativeAI\",\n",
" }\n",
")"
Copy link
Member

Choose a reason for hiding this comment

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

본 튜토리얼에서는 OpenAI API Key가 필요한 것이 아니라 GOOGLE_API_KEY이 필요한 것이다 보니 해당 부분을 GOOGLE_API_KEY 으로 바꾸는 것이 좋을 것 같다고 생각합니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

GOOGLE_API_KEY로 반영해두었습니다 감사합니다!

hj0302 added 2 commits January 2, 2025 17:35
[Team] Existing content development team3
[Title] Google Generative AI
[Version] revision
[Language] ENG
[Package] langsmith, langchain, langchin_core, langchain_google_genai

Langchain and Google Generative AI Tutorial
- A tutorial on using Langchain's ChatGoogleGenerativeAI class to access various generative models from Google AI.
@teddylee777 teddylee777 merged commit 883e7ec into LangChain-OpenTutorial:main Jan 2, 2025
1 of 2 checks passed
@teddylee777 teddylee777 added the highlight remarkable job to look around label Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
highlight remarkable job to look around
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants