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

intermediate_source/tensorboard_profiler_tutorial.py 번역 진행 #761

Conversation

dnd-qodqks
Copy link
Contributor

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

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

아직 번역되지 않은 'intermediate_source/tensorboard_profiler_tutorial.py' 문서를 번역하였습니다.

Copy link
Contributor

@jih0-kim jih0-kim left a comment

Choose a reason for hiding this comment

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

리뷰 진행하였습니다.

intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
@dnd-qodqks
Copy link
Contributor Author

리뷰 감사합니다! 말씀해주신 부분 수정해보았습니다 :)

Copy link
Member

@falconlee236 falconlee236 left a comment

Choose a reason for hiding this comment

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

문서 리뷰를 완료했습니다
전체적으로 용어를 한국어로 번역한 것이 있는데, 대부분은 잘 표현한 것 같지만 몇가지가 조금 어색한 것이 있어서 그것만 수정 부탁드렸습니다
긴 문서 번역하느라 고생 많으셨습니다

intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

긴문서 번역하느라 고생하셨습니다
전반적으로 잘되어있으나
번역자 이름 넣는 부분이 빠져있습니다
저자가 없어도 번역자는 넣어두니 다른 페이지 참고하셔서 추가 부탁드립니다.
그리고 다른 분들 피드백도 확인후 수정 부탁드립니다.

intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
@dnd-qodqks
Copy link
Contributor Author

falconlee236 말씀해주신 부분 모두 수정했습니다. 피드백해주셔서 감사합니다. :)

@dnd-qodqks
Copy link
Contributor Author

hyoyoung 피드백 감사합니다! 말씀해주신 부분 모두 수정하였습니다!!

Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

번역자 부분을 이름 넣는게 빠져있는거 같습니다

나머지 추가 피드백도 확인 부탁드립니다

intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

전반적으로 잘되어있습니다.

사소한 추가 피드백 확인부탁드립니다

intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
intermediate_source/tensorboard_profiler_tutorial.py Outdated Show resolved Hide resolved
@dnd-qodqks
Copy link
Contributor Author

추가 피드백 반역하였습니다. 감사합니다 :)

Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

good

@dnd-qodqks
Copy link
Contributor Author

@hyoyoung 지속적으로 피드백 주셔서 감사합니다 😊

@hyoyoung hyoyoung merged commit 139ae22 into PyTorchKorea:master Nov 26, 2023
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.

4 participants