-
Notifications
You must be signed in to change notification settings - Fork 282
04-MODEL / 08-HuggingFace-Pipeline #73
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
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.
안녕하세요, @sunworl 선월님! 배인진 입니다.
고생많으셨습니다.
아래 내용과 기타 의견 항목 확인부탁드립니다!
[Review]
- Review OS(Windows/Mac/Linux): Mac
- Template Rule 가이드를 준수 하였는가(YES/NO): YES
- Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): NO
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): 이미지 없음
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): NO
- 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요):
- 기타 의견
(1) base 브랜치와 conflict 발생하는 부분 해결이 필요합니다.
(2) 미팅로그 슬라이드 28장 안내 확인해보시면Jupyter Notebook 출력은 전부 출력이 나온 상태 그대로 commit 해 주세요
라고 쓰여있어서요. 각 cell의 출력 결과가 포함되어 있지 않은 상태라 추가해주시면 좋겠습니다.
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.
안녕하세요, @sunworl 선월님! 정민수 입니다.
늦게 리뷰 해드려 죄송합니다.
[Review]
Review OS(Windows/Mac/Linux): Windows
Template Rule 가이드를 준수 하였는가(YES/NO): YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): NO
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): 이미지 없음
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): NO
모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요):
기타 의견
인진님께서 꼼꼼하게 리뷰해주신 것 같습니다.
저도 추가적으로 발견한 부분들 첨부해드리겠습니다!
…8-HuggingFace-Pipeline.ipynb
…E-3] 04-MODEL /08-HuggingFace-Pipeline.ipynb
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.
@sunworl 선월님, 안녕하세요! 폴더를 새로 추가하신 부분에 대해 추가로 의견 드립니다.
그리고 이전에 말씀드렸던 부분들이 반영이 안되어 있습니다. 다시 한번 확인부탁드립니다!
- Table of Contents 링크 수정
- HuggingFacePipeline import문
- PyTorch 관련 의견 확인 부탁드립니다
- 기타 의견에 작성했던 부분 확인부탁드려요, Jupyter Notebook 출력을 포함해주셔야 해요.
미팅로그 슬라이드 28장 안내 확인해보시면 Jupyter Notebook 출력은 전부 출력이 나온 상태 그대로 commit 해 주세요 라고 쓰여있어서요. 각 cell의 출력 결과가 포함되어 있지 않은 상태라 추가해주시면 좋겠습니다.
@sunworl 선월님, 안녕하세요! 다른 부분은 수정된 내용 확인하고 resolve 했습니다 👍 |
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.
수고하셨습니다.
08-HuggingFace-Pipeline