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 constraints.md #163

Merged
merged 3 commits into from
Oct 30, 2019
Merged

Update constraints.md #163

merged 3 commits into from
Oct 30, 2019

Conversation

Denny-Hwang
Copy link
Contributor

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

#76

단어 논의 결과에 맞게 수정하였습니다.
세부 내용은 아래와 같습니다.

  • 소제목에 영문 원문 삽입
  • 인수 -> 인자
  • 레이어 -> 층
  • 히든 -> 은닉

@Denny-Hwang Denny-Hwang changed the title (ver 1) Update constraints.md Update constraints.md Oct 13, 2019
Copy link
Contributor

@Uwonsang Uwonsang left a comment

Choose a reason for hiding this comment

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

을 이용해서 주요 단어에 아래첨자만 추가해주시면 될 것 같습니다.
감사합니다.


`constraints` 모듈의 함수는 최적화 과정에서 네트워크 매개변수에 제약(예시. 비음수)을 설정할 수 있도록 합니다.
`constraints` 모듈의 함수는 최적화 과정에서 네트워크 매개변수에 제약(예시: 비음수)을 설정할 수 있도록 합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

비음수 -> 음이 아닌non-negativity
으로 맞춰주시면 될 것 같습니다.

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.

네 반영하겠습니다!


이러한 레이어들은 2가지 키워드 인수를 노출합니다:
이러한 층들은 2가지 키워드 인자를 노출합니다.
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.

네 감사합니다. 원문에서 arguments로 쓰여서 '인수'대신 '인자'로 바꿔서

-> '키워드 인자를 가집니다.' 로 수정하겠습니다.

@@ -71,16 +72,16 @@ keras.constraints.NonNeg()
keras.constraints.UnitNorm(axis=0)
```

히든 레이어 유닛에 대응하는 가중치가 단위 노름을 가지도록 제약합니다.
은닉 유닛에 대응하는 가중치가 단위 노름을 가지도록 제약합니다.
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.

원문에서 'hidden unit'으로 적혀 있었는데요, 이전 번역자분께서 '히든 레이어'로 번역하셨습니다. 그래서 우리 단어논의에 맞게 '은닉 유닛'으로 수정하였습니다. 이건 유지하도록 하겠습니다. 감사합니다.

@@ -98,10 +99,10 @@ keras.constraints.MinMaxNorm(min_value=0.0, max_value=1.0, rate=1.0, axis=0)

MinMaxNorm 가중치 제약.

히든 레이어에 대응하는 가중치를 제약해서
은닉 유닛에 대응하는 가중치를 제약해서
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.

원문에서 'hidden unit'으로 적혀 있었는데요, 이전 번역자분께서 '히든 레이어'로 번역하셨습니다. 그래서 우리 단어논의에 맞게 '은닉 유닛'으로 수정하였습니다. 이건 유지하도록 하겠습니다. 감사합니다.

@fuzzythecat
Copy link
Collaborator

감사합니다!

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