INTRODUCTION TO PYTORCH TENSORS 번역(#130)

josseong commented 8 months ago

라이선스 동의

변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래 [ ]를 [x]로 만들어주세요.

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

PR 종류

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

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

PR 설명

이 PR로 무엇이 달라지는지 대략적으로 알려주세요. 파이토치 튜토리얼중 Introduction to PyTorch on YouTube의 INTRODUCTION TO PYTORCH TENSORS문서를 번역하였습니다.

hyoyoung commented 8 months ago

[X ] 기여하기 문서를 확인하였으며, 본 PR 내용에 BSD 3항 라이선스가 적용됨에 동의합니다. [X ] 번역되지 않은 튜토리얼을 번역하는 기여

위 2부분의 X 표시옆에 공백이 있어서 체크가 제대로 안되고 있습니다 수정해서 공백제거 부탁드립니다

khleexv commented 5 months ago

앗 ???,,, 리뷰 도중 뭔가 이상해서 확인해보니 이미 번역되어 있는 문서인 것 같습니다. https://github.com/PyTorchKorea/tutorials-kr/blob/master/beginner_source/introyt/tensors_deeper_tutorial.py

@josseong 님께서 12월 18일 마스터 브랜치 기준으로 작업해주셨는데, 그 이전에 이미 번역 중인 PR이 있었고, 12월 26일에 마스터 브랜치에 해당 PR이 적용되어서, 번역이 진행 중인 것을 확인하지 못하신 것 같습니다 ㅠㅠ..

이런 경우에는 어떻게 하는 게 좋을까요? @hyoyoung

hyoyoung commented 5 months ago

앗 ???,,, 리뷰 도중 뭔가 이상해서 확인해보니 이미 번역되어 있는 문서인 것 같습니다. https://github.com/PyTorchKorea/tutorials-kr/blob/master/beginner_source/introyt/tensors_deeper_tutorial.py

@josseong 님께서 12월 18일 마스터 브랜치 기준으로 작업해주셨는데, 그 이전에 이미 번역 중인 PR이 있었고, 12월 26일에 마스터 브랜치에 해당 PR이 적용되어서, 번역이 진행 중인 것을 확인하지 못하신 것 같습니다 ㅠㅠ..

이런 경우에는 어떻게 하는 게 좋을까요? @hyoyoung

이경우는 열심히 번역해주셔서 죄송한일인데, 닫고 기존 번역 수정으로 하는게 어떨까 싶습니다

@josseong 번역하느라 고생하셨습니다. 확인하고 PR 닫아주시길 부탁드립니다 @khleexv 리뷰하느라 고생하셨습니다

khleexv commented 5 months ago

이와 별개로, 작업해주신 것에 피드백 드리자면, PR 본문에 이슈 번호가 잘못 지정되어 있습니다. PR 보내기 전에 이슈가 먼저 존재하는지 확인하고, 없으면 직접 열어주시면 될 것 같습니다.

그리고, 열심히 고쳐주셨지만, 아직도 문법 오류로 렌더링이 깨지는 부분이 일부 있습니다. CONTRIBUTING.md 문서 참고하셔서 직접 빌드해보시면, 문법 에러를 쉽게 찾을 수 있습니다. 어렵지 않으니, 한번 빌드해보시는 것도 나쁘지 않을 것 같습니다.

혹시나 작업 도중 어려움이 있으시면 discuss.pytorch.kr 에서 질문해주시면 도움 받으실 수 있으실 겁니다!

hyoyoung commented 2 months ago

2주 이상 피드백이 없어서 닫습니다 필요하면 다시 열어주시기 바랍니다

PyTorchKorea / tutorials-kr