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_source/introyt /captumyt.py 번역 #781

Closed
wants to merge 1 commit into from

Conversation

diligejy
Copy link

@diligejy diligejy commented Sep 8, 2023

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

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

번역되지 않았던 captum 튜토리얼을 번역하였습니다.

Copy link
Contributor

@ganghe74 ganghe74 left a comment

Choose a reason for hiding this comment

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

번역 리뷰 전에 코멘트 몇 개 남겨뒀는데, 참고하셔서 reStructuredText 문법 오류 수정 부탁드립니다.

===============================
**번역**: '송진영 <https://github.com/diligejy>'
Copy link
Contributor

Choose a reason for hiding this comment

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

문법 오류 있습니다. 줄 마지막에 _ 문자 추가하셔야 됩니다!


Captum에는 세 가지 속성(attribution)가 있습니다:

- **특징 속성(Feature Attribution)**는 특정 출력물을 생성한 입력물의 특징적인 측면에서 설명하고자 하며
Copy link
Contributor

Choose a reason for hiding this comment

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

**strong emphasis** 뒤에 ., ,, (공백) 같은 문자 아니면 렌더링이 제대로 안되는 것 같은데, 문서 전반적으로 ** 뒤에 한글이 있는 경우 띄어쓰기가 잘 안되어있습니다... ㅠㅠ

Copy link
Contributor

@ganghe74 ganghe74 Sep 9, 2023

Choose a reason for hiding this comment

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

렌더링 깨진 예시입니다.

chrome_Fly6NeG1q3
chrome_bngFnWICLs

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.

긴 문서 번역하느라 수고하셨습니다

몇가지 수정사항이 있는데 확인 부탁드립니다

Full documentation, an API reference, and a suite of tutorials on
specific topics are available at the `captum.ai <https://captum.ai/>`__
website.
Captum은 통합된 그레디언트(Integrated Gradients)를 포함한 최신 알고리즘을 제공하여 연구자와 개발자에게 어떤 기능이 모델의 출력에 속성하고 있는지 쉽게 파악할 수 있는 방법을 제공합니다.
Copy link
Member

Choose a reason for hiding this comment

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

여기서 feature는 속성하다기보다
모델이나 네트워크가 찾아낸 feature를 의미하므로 특징으로 바꾸는건 어떨까요?


- **퍼터베이션 기반 알고리즘(Perturbation-based algorithms)**은 입력의 변화에 대응하여 모델, 계층 또는 뉴런 출력의 변화를 확인합니다.
입력 퍼터베이션(input perturbations)은 일정한 방향을 가지거나 랜덤할 수 있습니다.
**폐색(Occlusion),** **특징 절제(Feature Ablation),** and **기능 순환(Feature Permutation)**은 모두 퍼터베이션 기반 알고리즘입니다.
Copy link
Member

Choose a reason for hiding this comment

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

vision에서는 occlusion을 가려짐으로 번역 가능할거 같습니다

# now.
#
# 이제 TorchVision 모델 라이브러리를 사용하여 사전학습된 ResNet을 다운로드해 보겠습니다.
# 우리는 학습을 시키지 않았기 때문에 일단 평가모드를 사용하겠습니다.
Copy link
Member

Choose a reason for hiding this comment

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

강조의 의미가 아닌경우 주어는 생략하면 좋을거 같습니다

transform_normalize = transforms.Normalize(
mean=[0.485, 0.456, 0.406],
std=[0.229, 0.224, 0.225]
)

transformed_img = transform(test_img)
input_img = transform_normalize(transformed_img)
input_img = input_img.unsqueeze(0) # the model requires a dummy batch dimension
input_img = input_img.unsqueeze(0) # 모형에 더미 배치 차원(dimension)이 필요함
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.

필요하지 않는 # 라인은 추가하지 않는게 좋을거 같습니다

@hyoyoung
Copy link
Member

2주 이상 피드백이 없어서 닫습니다 필요하면 다시 열어주시기 바랍니다

@hyoyoung hyoyoung closed this May 12, 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.

None yet

3 participants