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

Skip to content

Conversation

hellohotkey
Copy link
Contributor

[N-2] 07-Text Splitter / 01-CharacterTextSplitter

[N-2] 07-Text Splitter / 01-CharacterTextSplitter
@hellohotkey hellohotkey self-assigned this Jan 3, 2025
@hellohotkey
Copy link
Contributor Author

@kofsitho87 @fastjw 늦은 시간에 죄송합니다. 깃허브가 익숙치 않아 헤매다 보니 기한을 한참 넘겼습니다. 시간되실때 확인 부탁드립니다.

@hellohotkey hellohotkey changed the title Create 01-CharacterTextSplitter.ipynb [N-2] 07-Text Splitter / 01-CharacterTextSplitter Jan 3, 2025
@sohyunwriter
Copy link
Contributor

코드 뒷 부분이 날아간 것 같아요..! 주피터노트북 한 번 확인 부탁드려요.

[N-2] 07-Text Splitter / 01-CharacterTextSplitter
@hellohotkey
Copy link
Contributor Author

코드 뒷 부분이 날아간 것 같아요..! 주피터노트북 한 번 확인 부탁드려요.

@sohyunwriter 코드 뒷부분을 제가 빠뜨렸네요... 다시 추가했습니다!

@teddylee777 teddylee777 requested a review from musangk January 4, 2025 01:48
몇 가지 수정했습니다
1. 해당 챕터에서는 필요없는 패키지 삭제했습니다.

2. 환경세팅이 ## 이고, 나머지 목차들이 ### 여서 나머지 목차들이 환경세팅의 하위 목차인것처럼 되어 있어서 수정했습니다.

코드는 이상없습니다! 감사합니다.
Copy link
Contributor

@fastjw fastjw left a comment

Choose a reason for hiding this comment

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

시간이 없어서 제가 직접 수정했습니다!

  1. 필요없는 패키지 삭제했습니다.
  2. 환경세팅 목차 안에 나머지 내용들이 하위목차로 되어 있어서 목차 수정을 했어요!

코드는 이상없습니다!
고생많으셨습니다 :)

Copy link
Contributor

@fastjw fastjw left a comment

Choose a reason for hiding this comment

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

시간이 없어서 반영해두었어요

Copy link
Contributor

@kofsitho87 kofsitho87 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. 꼭 필요한 파일들만 제출 되었는가? 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed 에서 확인 부탁 드립니다.(YES/NO): YES

고생하셨습니다.

@fastjw fastjw removed the request for review from musangk January 4, 2025 03:10
@fastjw
Copy link
Contributor

fastjw commented Jan 4, 2025

  1. Review OS(Windows/Mac/Linux): MAC
  2. Template Rule 가이드를 준수 하였는가(YES/NO): YES
  3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
  4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): N/A
  5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
  6. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES
  7. 꼭 필요한 파일들만 제출 되었는가? 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed 에서 확인 부탁 드립니다.(YES/NO): YES

@hellohotkey
Copy link
Contributor Author

[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. 꼭 필요한 파일들만 제출 되었는가? 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed 에서 확인 부탁 드립니다.(YES/NO): YES

고생하셨습니다.

감사합니다 희웅님. 대시보드 리뷰 상태를 '완료'로 변경하겠습니다!

@teddylee777 teddylee777 merged commit 14ae724 into LangChain-OpenTutorial:main Jan 4, 2025
0 of 2 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.

5 participants