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

beginner/audio_io_tutorial 번역 #481

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hrxorxm
Copy link
Contributor

@hrxorxm hrxorxm commented Jul 2, 2022

라이선스 동의

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

관련 이슈 번호

PR 종류

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

PR 설명

beginner/audio 시리즈에서 audio io 튜토리얼을 번역하였습니다.

Copy link
Contributor

@Taeyoung96 Taeyoung96 left a comment

Choose a reason for hiding this comment

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

번역 하시느라 고생많으셨습니다!
전체적으로 Load라는 단어를 불러오기로 번역하는 것이 어떨지 제안드립니다. :)

@@ -255,21 +253,20 @@ def inspect_file(path):
print(metadata)

######################################################################
# Loading audio data into Tensor
# 텐서에 오디오 데이터 로드하기
Copy link
Contributor

Choose a reason for hiding this comment

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

로드라는 단어를 불러오기로 바꾸면 어떨까요?
텐서에 오디오 데이터 로드하기 >> 오디오 데이터 텐서로 불러오기.

# ------------------------------
#
# To load audio data, you can use ``torchaudio.load``.
# 오디오 데이터를 로드하기 위해, ``torchaudio.load`` 를 사용할 수 있습니다.
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.

오오 제가 왜 저렇게 했을까요. 알려주셔서 감사합니다!

@9bow
Copy link
Member

9bow commented Jul 9, 2022

어흑, 1.12에서 오디오 튜토리얼들에 변화가 많네요ㅠ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants