-
Notifications
You must be signed in to change notification settings - Fork 282
prototype_source/skip_param_init.rst 번역 #559
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
Conversation
vscode-hanspell을 활용하여 맞춤법 검사를 진행했습니다.
이슈번호 링크가 걸리지 않은 것 같습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일부분만 일단 수정 사항 남겼습니다. 나머지 부분은 나중에 시간 되면 다시 코멘트 달도록 하겠습니다!
prototype_source/skip_param_init.rst
Outdated
|
||
1. The module must accept a `device` kwarg in its constructor that is passed to any parameters | ||
or buffers created during construction. | ||
1. 모듈을 생성할 때 매개변수와 버퍼로 전달되는 모듈의 생성자 내 `device` keyword argument를 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
키워드 인자 (keyword argument)
로 번역하는건 어떨까요?
prototype_source/skip_param_init.rst
Outdated
2. The module must not perform any computation on parameters or buffers in its constructor except | ||
initialization (i.e. functions from `torch.nn.init`). | ||
2. 모듈은 초기화를 제외하고 모듈의 생성자 내 매개변수 또는 버퍼 계산을 수행하지 않아야 합니다 | ||
(i.e. `torch.nn.init`의 함수). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i.e. 는 "즉", "다시 말해서" 정도로 번역하면 좋을 것 같습니다.
|
||
# To ensure support for the meta device, avoid using ops except those in | ||
# torch.nn.init on parameters in your module's constructor. | ||
# meta 장치 지원을 확실히 하기 위해 모듈의 생성자 내 매개변수에 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위에서는 device 그대로 썼지만, 여기선 장치라고 번역이 되었으므로 장치 (device)라고 써주는 것이 좋을 것 같습니다.
prototype_source/skip_param_init.rst
Outdated
# Pass device along recursively. All submodules will need to support | ||
# them as well; this is the case for all torch.nn provided modules. | ||
# ==== 사례 2: 모듈의 서브 모듈을 생성합니다. ==== | ||
# 모든 서브 모듈은 device를 재귀적으로 전달해야 합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
두 문장을 하나로 합치신 것 같은데 세미콜론은 뒷 문장을 연결하는 역할이기 때문에 문장을 나누는게 좋을 것 겉습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전반적으로 매끄러운 번역입니다. 수고하셨습니다!
prototype_source/skip_param_init.rst
Outdated
# them as well; this is the case for all torch.nn provided modules. | ||
# ==== 사례 2: 모듈의 서브 모듈을 생성합니다. ==== | ||
# 장치 (device)를 재귀적으로 전달해야 합니다. 모든 서브 모듈 또한 해당 사항을 지원합니다. | ||
# torch.nn 에 제공되는 모든 모듈의 예시입니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(모든 서브 모듈이 해당 사항을 지원해야 하기 때문에) 장치를 재귀적으로 전달한다는 것인데 그런 의미가 잘 드러나지 않은 것 같습니다. 번역이 좀 까다로운 것 같은데... 아래처럼 번역하는 것을 조심스럽게 제안합니다.
'모든 서브 모듈이 해당 사항을 지원해야 하기 때문에 장치를 재귀적으로 전달합니다. 이는 torch.nn이 제공하는 모듈들의 경우에 해당합니다.'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
잘 번역하신 것 같습니다. 고생하셨습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전반적으로 읽기 편한 번역입니다. 몇가지 수정사항을 부탁드립니다
prototype_source/skip_param_init.rst
Outdated
-------------- | ||
|
||
모듈 구성 중 매개변수 초기화를 건너뛰게 되어 낭비되는 계산을 피할 수 있습니다. | ||
:func:`torch.nn.utils.skip_init` 함수를 사용하여 쉽게 실행할 수 있습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'함수를 사용하여 쉽게 실행할 수 있습니다.'보다 accomplished를 의미를 조금 더 살려보는 것도 좋을 듯 합니다.
'함수를 사용하면 쉽게 건너뛰기가 가능합니다.' 정도는 어떨까요?
prototype_source/skip_param_init.rst
Outdated
simply by adhering to these requirements: | ||
:func:`torch.nn.utils.skip_init` 의 구현(참고 :ref:`Details`) 방법에 따라, | ||
모듈이 함수와 호환되기 위한 두 가지 요구사항이 있습니다. | ||
다음의 요구사항을 이행하면 커스텀 모듈의 매개변수 초기화 스킵 기능을 선택할 수 있습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위에서는 초기화 스킵을 초기화 건너뛰기라고 번역했으니 여기도 맞추는게 좋을듯합니다
prototype_source/skip_param_init.rst
Outdated
The following example demonstrates a module updated to support the `device` | ||
kwarg by passing it along to any created parameters, buffers, or submodules: | ||
다음 예시는 작성된 매개변수, 버퍼 또는 서브 모듈에 `device` 키워드 인자 (keyword argument)를 전달하여 지원하도록 | ||
모듈이 업데이트된 것을 나타냅니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
원본 문장 자체가 복잡하니 쪼개서 번역하는 것도 방법일것 같습니다
The following example demonstrates a module updated to support the
device
kwarg by passing it along to any created parameters, buffers, or submodules
The following example demonstrates a module updated to support the
device
kwarg by passing
it along to any created parameters, buffers, or submodules
아래 예시는 모듈이 device
키워드 인자(keyword argument) 전달을 지원하기 위해 업데이트 된것을 보여줍니다.
생성된 파라미터, 버퍼, 서브모듈를 따라서.
합쳐보면
다음은 device
키워드 인자(keyword argument)가 생성된 파라미터, 버퍼, 서브모듈로 따라 전달되기 위한
모듈 업데이트를 보여주는 예시입니다.
prototype_source/skip_param_init.rst
Outdated
It works by instantiating the module onto a "meta" device, which has tensor shape information | ||
but does not allocate any storage. The `torch.nn.init` ops are specially implemented for this meta device | ||
so that they have no-op behavior. This results in the parameter intialization logic being essentially skipped. | ||
모듈은 "meta" 장치 (device)로 인스턴스화하여 동작합니다. tensor shape 정보를 가지고 있지만 스토리지는 할당하지 않습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
스토리지는 저장 공간으로 바꿔써도 좋을듯합니다
prototype_source/skip_param_init.rst
Outdated
|
||
Note that this pattern only works for modules that properly support a `device` kwarg during construction, as | ||
described in :ref:`Updating`. | ||
:ref:`Updating` 에 설명된 대로 이 패턴은 모듈 구성 중 `device` 키워드 인자 (keyword argument)를 적절히 지원하는 모듈에서만 작동합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
키워드 인자(keyword argument)는 여러차례 나오므로
맨 처음 이외에는 그냥
키워드 인자라고만 써도 될것 같습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good
vscode-hanspell을 활용하여 맞춤법 검사를 진행했습니다.
라이선스 동의
변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래
[ ]
를[x]
로 만들어주세요.관련 이슈 번호
이 Pull Request와 관련있는 이슈 번호를 적어주세요.
이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )
PR 종류
이 PR에 해당되는 종류 앞의
[ ]
을[x]
로 변경해주세요.PR 설명
이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
prototype_source/skip_param_init.rst 파일을 영문에서 한문으로 수정합니다.
https://tutorials.pytorch.kr/prototype/skip_param_init.html 페이지가 한글화됩니다.
연습용 레포에서 피드백 받은 내용입니다. (https://github.com/hyoyoung/22-tutorials-ex/pull/23)
피드백 내용을 반영하였습니다.
vscode-hanspell을 활용하여 추가적으로 맞춤법 검사를 진행했습니다.
make html-noplot 빌드하여 최종 웹화면을 확인했습니다.
device의 번역 관련하여 매개변수로 활용되는 경우 device로 번역하지 않았으며,
CPU device -> CPU 장치 와 같이 의미적으로 활용되는 경우 장치로 번역하였습니다.