-
Notifications
You must be signed in to change notification settings - Fork 282
[N-2] 07-Text Splitter / 01-CharacterTextSplitter #108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
[N-2] 07-Text Splitter / 01-CharacterTextSplitter
@kofsitho87 @fastjw 늦은 시간에 죄송합니다. 깃허브가 익숙치 않아 헤매다 보니 기한을 한참 넘겼습니다. 시간되실때 확인 부탁드립니다. |
코드 뒷 부분이 날아간 것 같아요..! 주피터노트북 한 번 확인 부탁드려요. |
[N-2] 07-Text Splitter / 01-CharacterTextSplitter
@sohyunwriter 코드 뒷부분을 제가 빠뜨렸네요... 다시 추가했습니다! |
몇 가지 수정했습니다 1. 해당 챕터에서는 필요없는 패키지 삭제했습니다. 2. 환경세팅이 ## 이고, 나머지 목차들이 ### 여서 나머지 목차들이 환경세팅의 하위 목차인것처럼 되어 있어서 수정했습니다. 코드는 이상없습니다! 감사합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
시간이 없어서 제가 직접 수정했습니다!
- 필요없는 패키지 삭제했습니다.
- 환경세팅 목차 안에 나머지 내용들이 하위목차로 되어 있어서 목차 수정을 했어요!
코드는 이상없습니다!
고생많으셨습니다 :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
시간이 없어서 반영해두었어요
There was a problem hiding this 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): Mac
- Template Rule 가이드를 준수 하였는가(YES/NO): YES
- Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): YES
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
- 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES
- 꼭 필요한 파일들만 제출 되었는가? 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed 에서 확인 부탁 드립니다.(YES/NO): YES
고생하셨습니다.
|
감사합니다 희웅님. 대시보드 리뷰 상태를 '완료'로 변경하겠습니다! |
[N-2] 07-Text Splitter / 01-CharacterTextSplitter