feat: translate fx profiling tutorial #767 #787
Conversation
|
아마 권한이 없으셔서, 이미 제출한 PR이 삭제가 되는지 모르겠네요. |
|
해결된 Feedback들은 보기 편하게 Resolved 상태로 변경해두는 게 좋을 것 같습니다. |
|
@khleexv @hyoyoung 확인 부탁드립니다 |
|
@Ssunbell PR 화면에서 오른쪽 위에 'Reviewers'보시면 지난번에 리뷰한 사람들이 있습니다 |
| # performance. That is, for the following invocation, which parts | ||
| # of the model are taking the longest? | ||
| # 이제 모델을 불러왔으므로, 모델의 성능을 좀더 깊이 조사하고자 합니다. | ||
| # 즉, 다음 호출에 대해서, 모델의 어떤 부분이 가장 오래 걸립니까? |
There was a problem hiding this comment.
'That is'는 아마도 이전 문장의 'inspect deeper'를 지칭하는 뜻일거 같습니다.
그렇다면 '즉'보다는 '그거라면'과 비슷할텐데.
'즉'이나 '그거라면' 둘중 어떤걸 넣어도 어색하니까 차라리 아예 빼서
'다음 호출에 대해서, 모델의 어떤 부분이 가장 오래 걸립니까?'
이렇게 바꿔보는건 어떨까요?
|
|
||
| ###################################################################### | ||
| # Capturing the Model with Symbolic Tracing | ||
| # |
| # the definition of our model in a data structure we can manipulate | ||
| # and examine. | ||
| # 다음으로, FX의 Symbolic Tracing 메커니즘을 활용하여 우리가 조작하고 | ||
| # 조사할 수 있는 자료구조에서 우리 모델의 정의를 포착할 것입니다. |
There was a problem hiding this comment.
여기도 '우리 모델'라는 표현보다는 '해당 모델'의 정의로 하거나
'모델'의 정의를 포착한다는 걸로 바꾸는게 더 일관성있을거 같습니다
| # on each node) represent the values passed between these call-sites. More | ||
| # information about the Graph representation and the rest of FX's APIs ca | ||
| # be found at the FX documentation https://pytorch.org/docs/master/fx.html. | ||
| # 이것은 ResNet18 모델의 그래프 표현을 제공합니다. |
There was a problem hiding this comment.
주어에서 '이것은' 이라고 하면 조금 어색한듯합니다
'해당 부분은' 이라거나 '이 코드에서는'이라고 바꾸는건 어떨까요?
| # install the profiling behavior we want. The goal is to have an object to which | ||
| # we can pass a model, invoke the model 1 or more times, then get statistics about | ||
| # how long the model and each part of the model took during those runs. | ||
| # ``Interpreter`` 로부터 상속받음으로써, 다양한 기능을 덮여씌울 수 있고 |
There was a problem hiding this comment.
'덮여씌울' -> '덮어씌울'의 오타인 것 같습니다 확인 부탁드립니다.
Ssunbell
deleted the
feat/translate-intermediate_source-fx-profiling-tutorial
branch
라이선스 동의
변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래
[ ]를[x]로 만들어주세요.관련 이슈 번호
이 Pull Request와 관련있는 이슈 번호를 적어주세요.
이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )
PR 종류
이 PR에 해당되는 종류 앞의
[ ]을[x]로 변경해주세요.PR 설명
intermediate_source/fx_profiling_tutorial.py 한글 번역