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

Skip to content

Conversation

johnny9210
Copy link
Contributor

[Team] New Content Development Team 2

[Title] [N-2] 03-TextSplitter / 03-TokenTextSplitter [Version] initial
[Language] ENG
[Packages] langchain, langchain_text_splitters, tiktoken, spacy, sentence-transformers, nltk, konlpy

Included a tutorial on TokenTextSplitter, covering basic usage, advanced token-based splitting techniques, and custom configuration options for optimized text segmentation.

리뷰 다시 한번 부탁드립니다.. closed가 되면서 다시 pr드립니다.

[Team] New Content Development Team 2

[Title] [N-2] 03-TextSplitter / 03-TokenTextSplitter
[Version] initial
[Language] ENG
[Packages] langchain, langchain_text_splitters, tiktoken, spacy, sentence-transformers, nltk, konlpy

Included a tutorial on TokenTextSplitter, covering basic usage, advanced token-based splitting techniques, and custom configuration options for optimized text segmentation.
BaBetterB
BaBetterB previously approved these changes Jan 3, 2025
Copy link
Contributor

@BaBetterB BaBetterB left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

package.install 에서 verbose 옵션을 False 로 주어 출력을 silent 하였습니다.
load_dotenv(override=True) 로 주었습니다.
Copy link
Contributor

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

verbose=False 옵션으로 설치 출력 메시지를 silent 하였습니다.
load_dotenv(override=True) 로 주었습니다.
Peer Reviewer 를 지정하였습니다.

@teddylee777 teddylee777 merged commit bc9ff05 into LangChain-OpenTutorial:main Jan 3, 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.

3 participants