polynomial_autograd.py 추가 오탈자 수정#1125
Open
rlarbdus128-sys wants to merge 2 commits into
Open
Conversation
5 tasks
Member
|
@rlarbdus128-sys |
Contributor
Author
conflict 해결 완료했습니다! 확인 부탁드립니다! |
jih0-kim
requested changes
Jun 8, 2026
jih0-kim
left a comment
jih0-kim
left a comment
Member
There was a problem hiding this comment.
용어집 표기 반영에 감사드립니다. 몇 가지 코멘트 확인 부탁드려요.
Comment on lines
+22
to
+23
| # 입력값과 출력값을 갖는 Tensor들을 생성합니다. | ||
| # requires_grad=False가 기본값으로 설정되어 역전파 단계 중에 이 Tensor들에 대한 변화도를 |
Member
There was a problem hiding this comment.
Tensor들 이라고 복수형으로 표현하는 것이 더 어색해 Tensor 라고만 표현해도 될 것 같습니다.
L39, 42, 50 등 모두 검토해주세요.
| y = torch.sin(x) | ||
|
|
||
| # 가중치를 갖는 임의의 텐서를 생성합니다. 3차 다항식이므로 4개의 가중치가 필요합니다: | ||
| # 가중치를 갖는 임의의 Tensor를 생성합니다. 3차 다항식이므로 4개의 가중치가 필요합니다: |
Member
There was a problem hiding this comment.
: 대신 . 로 표현하는 게 더 자연스러울 것 같습니다.
| # 이때 손실은 (1,) shape를 갖는 텐서입니다. | ||
| # loss.item()으로 손실이 갖고 있는 스칼라 값을 가져올 수 있습니다. | ||
| # Tensor들간의 연산을 사용하여 손실(loss)을 계산하고 출력합니다. | ||
| # 이 때 손실은 (1,) shape을 갖는 Tensor입니다. |
Member
There was a problem hiding this comment.
이때는 붙여 쓰는 것이 맞는 표기로, 수정 요청드립니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
라이선스 동의
변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래
[ ]를[x]로 만들어주세요.관련 이슈 번호
이 Pull Request와 관련있는 이슈 번호를 적어주세요.
이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )
PR 종류
이 PR에 해당되는 종류 앞의
[ ]을[x]로 변경해주세요.PR 설명
이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
TRANSLATION_GUIDE.md에 따라 line 3, 9, 12, 13, 22, 23, 28, 30, 39, 42, 43, 50, 52에서 '텐서'로 번역된 부분을 Tensor로 수정하였습니다.