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

torch_compile_backend_ipex.rst 번역 #901

Merged

Conversation

jh941213
Copy link
Contributor

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

문서번역

Copy link
Contributor

@hadh93 hadh93 left a comment

Choose a reason for hiding this comment

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

번역하시느라 고생 많으셨습니다!
몇 가지 제안 드리니 확인 부탁드립니다 🙏

It targets to improve hardware resource usage efficiency on Intel platforms for better performance.
The `ipex` backend is implemented with further customizations designed in Intel® Extension for
PyTorch* for the model compilation.
`torch.compile`을 통해 더 원활한 작동을 위해, Intel® Extension for PyTorch는 ``ipex``라는 백엔드를 구현했습니다. 이 백엔드는 Intel 플랫폼에서 하드웨어 자원 사용 효율성을 개선하여 성능을 향상시키는 것을 목표로 합니다. `ipex` 백엔드는 모델 컴파일을 위한 Intel® Extension for PyTorch에 설계된 추가 커스터마이징을 통해 구현되었습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. 마크업 문법이 적용되도록 띄어쓰기가 필요합니다.
  • torch.compile을 -> torch.compile
  • ipex라는 -> ipex 라는
  1. To work better with을 ~과 더 잘 작동하도록 으로 번역하면 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니다. 확인했습니다!

@@ -96,7 +92,7 @@ Check the example below to learn how to utilize the `ipex` backend with `torch.c
optimizer = torch.optim.SGD(model.parameters(), lr = LR, momentum=0.9)
model.train()

#################### code changes ####################
#################### 코드 변경 부분 ####################
import intel_extension_for_pytorch as ipex

# Invoke the following API optionally, to apply frontend optimizations
Copy link
Contributor

@hadh93 hadh93 Sep 8, 2024

Choose a reason for hiding this comment

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

코드안의 다른 주석도 번역이 필요할 것 같아요!

  • Invoke the following API optionally, to apply frontend optimizations

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니다. 확인했습니다!

@@ -130,8 +126,8 @@ Check the example below to learn how to utilize the `ipex` backend with `torch.c

#################### code changes ####################
Copy link
Contributor

@hadh93 hadh93 Sep 8, 2024

Choose a reason for hiding this comment

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

코드안의 다른 주석도 번역이 필요할 것 같아요!

  • #################### code changes ####################

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니다. 확인했습니다!

--------------

Check the example below to learn how to utilize the `ipex` backend with `torch.compile` for model inference with FP32 data type.
아래 예시를 통해 `ipex` 백엔드를 `torch.compile`와 함께 활용하여 FP32 데이터 유형으로 모델을 추론하는 방법을 알아보세요.
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. 빌드해 보니, 아래와 같은 글자 깨짐이 있습니다.
    image
  • 'FP32' 앞의 공백을 지웠다가 다시 입력해보시는 건 어떨까요?
  1. 마크업 문법이 적용되도록 띄어쓰기가 필요합니다.
  • torch.compile와 -> torch.compile

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.

전반적으로 잘되어있는데 몇가지 수정 사항이 필요합니다.
확인 후에 수정 부탁드립니다

It targets to improve hardware resource usage efficiency on Intel platforms for better performance.
The `ipex` backend is implemented with further customizations designed in Intel® Extension for
PyTorch* for the model compilation.
`torch.compile` 과 더 잘 작동하도록, Intel® Extension for PyTorch는 ``ipex`` 라는 백엔드를 구현했습니다. 이 백엔드는 Intel 플랫폼에서 하드웨어 자원 사용 효율성을 개선하여 성능을 향상시키는 것을 목표로 합니다. `ipex` 백엔드는 모델 컴파일을 위한 Intel® Extension for PyTorch에 설계된 추가 커스터마이징을 통해 구현되었습니다.
Copy link
Member

Choose a reason for hiding this comment

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

꼭 맞추지 않아도 되나 원문의 줄내림과 비슷한 정도로 줄내림 부탁드립니다

It targets to improve hardware resource usage efficiency on Intel platforms for better performance.
The `ipex` backend is implemented with further customizations designed in Intel® Extension for
PyTorch* for the model compilation.
`torch.compile` 과 더 잘 작동하도록, Intel® Extension for PyTorch는 ``ipex`` 라는 백엔드를 구현했습니다. 이 백엔드는 Intel 플랫폼에서 하드웨어 자원 사용 효율성을 개선하여 성능을 향상시키는 것을 목표로 합니다. `ipex` 백엔드는 모델 컴파일을 위한 Intel® Extension for PyTorch에 설계된 추가 커스터마이징을 통해 구현되었습니다.
Copy link
Member

Choose a reason for hiding this comment

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

ipex 백엔드는 모델 컴파일을 위한 Intel® Extension for PyTorch에 설계된 추가 커스터마이징을 통해 구현되었습니다.

위 문장은 조금 복잡해보입니다.

모델 컴파일을 위한 Intel® Extension for PyTorch에 설계된 추가 커스터마이징을 통해,
ipex` 백엔드가 구현되었습니다.

이정도로 바꾸던지 아니면 아예 문장은 분리해서 2문장으로 나누어도 좋을 것 같습니다

----------

Check the example below to learn how to utilize the `ipex` backend with `torch.compile` for model training with FP32 data type.

아래 예제를 통해, 여러분은 FP32 데이터 유형으로 모델을 학습할 때 `torch.compile` 과 함께 `ipex` 백엔드를 사용하는 방법을 배울 수 있습니다.
Copy link
Member

Choose a reason for hiding this comment

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

data type을 데이터 유형으로 번역하신 듯한데,
자료형이나 데이터 타입(data type)중에서 익숙한 단어로 바꿔보시는건 어떨까요?

import intel_extension_for_pytorch as ipex

# Invoke the following API optionally, to apply frontend optimizations
# 선택적으로 다음 API를 호출하여 프론트엔드 최적화를 적용합니다.
Copy link
Member

Choose a reason for hiding this comment

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

중간에 쉼표 구분을 살려주면 더 좋을 것 같습니다.

다음 API를 선택적으로 호출하여, 프론트엔드 최적화를 적용합니다.

@oh5221
Copy link
Contributor

oh5221 commented Sep 21, 2024

고생하셨습니다!
원문과 같이 보다 보니 한 가지 점이 눈에 보이네요.
확인 부탁드리겠습니다 ㅎㅎ

@jh941213
Copy link
Contributor Author

전반적으로 잘되어있는데 몇가지 수정 사항이 필요합니다. 확인 후에 수정 부탁드립니다

@hyoyoung 감사합니다. 쉼표등 생각을 못하고 있었네요. 수정 반영하겠습니다

@NK590
Copy link
Contributor

NK590 commented Sep 22, 2024

다른 분들이 좋은 리뷰 많이 남겨주셔서 크게 수정해야 될 부분은 보이지 않는 것 같습니다. 수고하셨습니다!

@Angela-Park-JE
Copy link
Contributor

잘 되어있는 번역에 피드백까지 반영된 후여서인지 저에게는 좋은 번역으로 보입니다.
원문의 의미를 잘 담은 명확하고 간결한 단어 선택이 정말 좋았습니다. 수고하셨습니다! 👍

@@ -158,7 +156,7 @@ Check the example below to learn how to utilize the `ipex` backend with `torch.c
#################### code changes ####################
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
Member

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.

감사합니다.

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.

�몇가지 수정 사항이 있습니다 확인 부탁드립니다.

import intel_extension_for_pytorch as ipex

# Invoke the following API optionally, to apply frontend optimizations

Copy link
Member

Choose a reason for hiding this comment

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

줄 앞에 빈 공백이 있습니다

Copy link
Member

Choose a reason for hiding this comment

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

여기 공백라인은 앞에 space가 있는거 같은데 그 space를 제거해주셔야합니다.

@@ -158,7 +156,7 @@ Check the example below to learn how to utilize the `ipex` backend with `torch.c
#################### code changes ####################
Copy link
Member

Choose a reason for hiding this comment

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

여기도 수정 부탁드립니다

It targets to improve hardware resource usage efficiency on Intel platforms for better performance.
The `ipex` backend is implemented with further customizations designed in Intel® Extension for
PyTorch* for the model compilation.
- `torch.compile` 과 더 잘 작동하도록, Intel® Extension for PyTorch는 ``ipex`` 라는 백엔드를 구현했습니다.
Copy link
Member

Choose a reason for hiding this comment

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

역자 정보 추가 부탁드립니다.

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

@hyoyoung hyoyoung merged commit 10f1fab into PyTorchKorea:master Oct 15, 2024
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.

6 participants