Skip to content

intermediate_source /per_sample_grads.py 번역 완료#1127

Open
NamsanMan wants to merge 1 commit into
PyTorchKorea:masterfrom
NamsanMan:translate-per-sample-grads
Open

intermediate_source /per_sample_grads.py 번역 완료#1127
NamsanMan wants to merge 1 commit into
PyTorchKorea:masterfrom
NamsanMan:translate-per-sample-grads

Conversation

@NamsanMan
Copy link
Copy Markdown
Contributor

라이선스 동의

변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.

더 자세한 내용은 기여하기 문서를 참고해주세요.

동의하시면 아래 [ ][x]로 만들어주세요.

  • 기여하기 문서를 확인하였으며, 본 PR 내용에 BSD 3항 라이선스가 적용됨에 동의합니다.

관련 이슈 번호

이 Pull Request와 관련있는 이슈 번호를 적어주세요.

이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )

PR 종류

이 PR에 해당되는 종류 앞의 [ ][x]로 변경해주세요.

  • 오탈자를 수정하거나 번역을 개선하는 기여
  • 번역되지 않은 튜토리얼을 번역하는 기여
  • 공식 튜토리얼 내용을 반영하는 기여
  • 위 종류에 포함되지 않는 기여

PR 설명

이 PR로 무엇이 달라지는지 대략적으로 알려주세요.

intermediate_source /per_sample_grads.py 의 내용을 번역 완료하였습니다.

@rlarbdus128-sys
Copy link
Copy Markdown

우선 전반적으로 이해하기 쉽게 정말 잘 번역하신 것 같습니다! 수고하셨습니다.

아래에, 제가 읽으며 수정을 제안드리고 싶은 부분을 간단히 적어보았으니 오탈자 PR 생성에 참고하셔도 좋을 것 같습니다.

line 6: 의미 전달을 위해 원문대로 의문문 문말에 '?'을 붙여도 좋을 것 같습니다.

line 149, 181: 표준어 규정에 따라 소괄호와 단어를 띄어쓰는 것보다 문맥에 맞게 붙여쓰는 것이 적합할 것으로 생각됩니다. 다른 line에서는 잘 지켜진 사항이기에 의도하신 부분인지 확인 후 수정해주시길 바랍니다.

line 99, 100, 116, 131, 143, 144, 149, 150, 151, 153, 163, 170, 173, 179, 212, 215, 217: 백틱 외부 띄어쓰기는 표준어 규정에 따르므로, 백틱과 조사 사이 띄어쓰기를 삭제하는 것이 적합할 것으로 보입니다. 다만 다른 한글화 문서를 보았을 때 가시성을 위해 백틱과 뒤따르는 단어를 띄어쓰는 것이 널리 허용되는 면이 있고 'PyTorch Korea 한국어 튜토리얼 번역 가이드라인' 등을 조사해보니 마땅한 규정이 없는 것으로 보입니다. 온전히 오탈자로 보기 어려우나 여지가 있으므로 멘토님께 여쭤보고 추가 수정을 할지 결정해보면 좋을 것 같습니다.

다시 한 번 정말 수고 많으셨습니다. 감사합니다.

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.

2 participants