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

Skip to content

Conversation

krkrong
Copy link
Contributor

@krkrong krkrong commented Jan 1, 2025

[N-3] 08-Embedding / 07-GPT4ALLEmbedding

기존 튜토리얼 내용과 크게 변동사항 없습니다.
리뷰 부탁드립니다!

[N-3] 08-Embedding / 06-GPT4ALLEmbedding
@krkrong krkrong self-assigned this Jan 1, 2025
LEE1026icarus
LEE1026icarus previously approved these changes Jan 2, 2025
@krkrong krkrong changed the title [N-3] 08-Embedding / 06-GPT4ALLEmbedding [N-3] 08-Embedding / 07-GPT4ALLEmbedding Jan 2, 2025
Copy link
Contributor

@LEE1026icarus LEE1026icarus left a comment

Choose a reason for hiding this comment

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

[Review]

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

@teddylee777
Copy link
Contributor

@LEE1026icarus

아래 가이드 맞춰 리뷰 작성 요청 드립니다!

[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. 기타 의견:

Copy link
Contributor

@dancing-with-coffee dancing-with-coffee left a comment

Choose a reason for hiding this comment

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

macos에서 정상 동작합니다! 맨 처음에 시작하는 Peer Review 파트에 제 이름 변경했다가 나중에 한번에 되지 않을까 해서 추가 commit은 안하고 review만 완료합니다!

[Review]

Review OS(Windows/Mac/Linux): Mac
Template Rule 가이드를 준수 하였는가(YES/NO): YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): 이미지 없음
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
모든 코드가 동작에 오류 없이 동작하는가? YES

@teddylee777 teddylee777 merged commit 81acbaa into LangChain-OpenTutorial:main Jan 3, 2025
3 of 4 checks passed
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