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

Change the content in 'layers/wrappers' #141

Merged
merged 5 commits into from
Oct 15, 2019

Conversation

airplane2230
Copy link
Contributor

@airplane2230 airplane2230 commented Oct 2, 2019

#98
Change the content in 'layers/wrappers'

Change the content in 'layers/wrappers'
None의 경우, 아웃풋이 합쳐지지 않고,
리스트로 반환됩니다.
{'sum', 'mul', 'concat', 'ave', None} 중 하나를 인수로 받습니다.
None의 경우, 아웃풋이 합쳐지지 않고, 리스트로 반환됩니다.
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.

감사합니다.
내용은 문제 없는 것 같네요. 그럼 단어까지 수정해서 이 pr에 다시 업데이트시키곘습니다~


인풋은 적어도 3D 이상이어야 하며, 그 중 색인 1의 차원은
시간 차원으로 인식합니다.
입력 값은 최소 3D의 형태를 가져야 하며, 그 중 첫 번째 색인은 시간 차원을 나타냅니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

index 1을 첫번째 색인이라고 하면 조금 헷갈릴 수 있을 것 같아요.
원문을 index 1(second element)로 수정하고,
-> index 1(두번째 원소)는 시간 차원을 나타냅니다
로 적는 것이 좀 더 명확할 것 같아요!

Copy link
Contributor

@Naruu Naruu left a comment

Choose a reason for hiding this comment

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

worklog의 단어집을 참고해서, 영어 단어를 한국어로 바꾸어주세요!
그리고 처음 번역된 단어들은 옆에 작게 적어주세요 :)
예)
layer -> 층Layer
#146
를 참고하시면 좋을 것 같아요!

airplane2230 and others added 3 commits October 6, 2019 17:34
지적해주신 단어를 변경하였습니다.
인수 -> 인자
+ recurrent --> RNN
+ `Bidirectional` 층에 사용할 초기 가중치
@fuzzythecat
Copy link
Collaborator

감사합니다!

@fuzzythecat fuzzythecat merged commit ee77a6d into keras-team:master Oct 15, 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