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

Skip to content

Conversation

ro-jun
Copy link
Contributor

@ro-jun ro-jun commented Dec 30, 2024

[Title] OpenAI Embeddings
[Version] initial
[Language] ENG
[Packages] langchain_openai, scikit-learn, matplotlib

  • Added available embedding dimension
  • Enhanced PCA visualization (documents and query)

[Version] initial
[Language] ENG
[Packages] langchain_openai, scikit-learn, matplotlib

- Added available embedding dimension
- Enhanced PCA visualization (documents and query)
@ro-jun ro-jun requested review from namyoungkim and acho98 December 30, 2024 21:22
Copy link
Contributor

@acho98 acho98 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
  7. 기타 의견: 없음

Copy link
Contributor

@namyoungkim namyoungkim 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
  7. 기타 의견: 없습니다. 고생하셨습니다~!

@teddylee777 teddylee777 merged commit 56714d7 into LangChain-OpenTutorial:main Jan 2, 2025
1 check failed
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