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

Skip to content

Conversation

sunworl
Copy link
Contributor

@sunworl sunworl commented Jan 1, 2025

08-HuggingFace-Pipeline

@sunworl sunworl requested review from ivybae and effort-type January 2, 2025 01:15
@sunworl sunworl self-assigned this Jan 2, 2025
Copy link
Contributor

@ivybae ivybae left a comment

Choose a reason for hiding this comment

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

안녕하세요, @sunworl 선월님! 배인진 입니다.
고생많으셨습니다.
아래 내용과 기타 의견 항목 확인부탁드립니다!

[Review]

  1. Review OS(Windows/Mac/Linux): Mac
  2. Template Rule 가이드를 준수 하였는가(YES/NO): YES
  3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): NO
  4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): 이미지 없음
  5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): NO
  6. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요):
  7. 기타 의견
    (1) base 브랜치와 conflict 발생하는 부분 해결이 필요합니다.
    (2) 미팅로그 슬라이드 28장 안내 확인해보시면 Jupyter Notebook 출력은 전부 출력이 나온 상태 그대로 commit 해 주세요 라고 쓰여있어서요. 각 cell의 출력 결과가 포함되어 있지 않은 상태라 추가해주시면 좋겠습니다.

Copy link
Member

@effort-type effort-type left a 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 발생시 코멘트에 적어주세요):
기타 의견

인진님께서 꼼꼼하게 리뷰해주신 것 같습니다.

저도 추가적으로 발견한 부분들 첨부해드리겠습니다!

Copy link
Contributor

@ivybae ivybae left a comment

Choose a reason for hiding this comment

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

@sunworl 선월님, 안녕하세요! 폴더를 새로 추가하신 부분에 대해 추가로 의견 드립니다.

그리고 이전에 말씀드렸던 부분들이 반영이 안되어 있습니다. 다시 한번 확인부탁드립니다!

미팅로그 슬라이드 28장 안내 확인해보시면 Jupyter Notebook 출력은 전부 출력이 나온 상태 그대로 commit 해 주세요 라고 쓰여있어서요. 각 cell의 출력 결과가 포함되어 있지 않은 상태라 추가해주시면 좋겠습니다.

@ivybae
Copy link
Contributor

ivybae commented Jan 4, 2025

@sunworl 선월님, 안녕하세요! 다른 부분은 수정된 내용 확인하고 resolve 했습니다 👍
아래 2가지 내용은 아직 더 수정이 필요해 보여 다시 확인 부탁드립니다~

@sunworl sunworl requested a review from fastjw January 4, 2025 07:36
Copy link
Member

@effort-type effort-type left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

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

4 participants