Skip to content
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

Update utils.md #150

Merged
merged 5 commits into from
Oct 19, 2019
Merged

Update utils.md #150

merged 5 commits into from
Oct 19, 2019

Conversation

Denny-Hwang
Copy link
Contributor

@Denny-Hwang Denny-Hwang commented Oct 8, 2019

#66

  • 2차리뷰어(휘용님)의 의견을 반영하여 수정하였습니다.

  • '그림'->'이미지'로 다시 환원하였습니다.

  • 인수->인자(10/13)

@Denny-Hwang Denny-Hwang mentioned this pull request Oct 11, 2019
Copy link
Contributor

@airplane2230 airplane2230 left a comment

Choose a reason for hiding this comment

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

마무리 단계네요. 고생하셨습니다.

sources/utils.md Outdated
@@ -131,12 +131,12 @@ __인수__
- __y__: 행렬로 변환할 클래스 벡터
(0부터 num_classes까지의 정수).
- __num_classes__: 클래스의 총 개수.
- __dtype__: 문자열로 표현된 인풋의 데이터 자료형
- __dtype__: 문자열로 표현된 입력의 데이터 자료형
Copy link
Contributor

Choose a reason for hiding this comment

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

입력의 -> 입력값의

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 반영하였습니다.

sources/utils.md Outdated
(`float32`, `float64`, `int32`...)

__반환값__

인풋의 이진행렬 표현.
입력 값의 이진행렬 표현.
Copy link
Contributor

Choose a reason for hiding this comment

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

입력 값의 -> 입력값의

헷갈렸습니다. 띄어쓰기를 안하는게 맞는 표현이더라구요 ㅎㅎ;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 수정하였습니다.

@fuzzythecat
Copy link
Collaborator

감사합니다!

@fuzzythecat fuzzythecat merged commit e088b29 into keras-team:master Oct 19, 2019
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