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

HardNet 번역 #83

Merged
merged 5 commits into from
Apr 2, 2023
Merged

HardNet 번역 #83

merged 5 commits into from
Apr 2, 2023

Conversation

cpprhtn
Copy link
Contributor

@cpprhtn cpprhtn commented Sep 5, 2022

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

  • 오탈자를 수정하거나 번역을 개선하는 기여
  • 번역되지 않은 모델 소개를 번역하는 기여
  • 공식 허브의 내용을 반영하는 기여
  • 위 종류에 포함되지 않는 기여

PR 설명

HardNet 번역

@KyubumShin
Copy link
Contributor

큰 문제 없이 잘 번역된것 같습니다!

w/ or w/o Depthwise Separable Conv respectively.
Their 1-crop error rates on imagenet dataset with pretrained models are listed below.
아래에는 각각 깊이별 분리 가능한 Conv 레이어가 있거나 없는 39, 68, 85개의 레이어를 포함한 4가지 버전의 HardNet 모델이 있습니다.
사전 훈련된 모델이 있는 ImageNet 데이터셋에서 1-crop 오류율은 아래에 나열되어 있습니다.
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.

문장이 조금 더 깔끔해질것 같습니다!

import urllib
url, filename = ("https://github.com/pytorch/hub/raw/master/images/dog.jpg", "dog.jpg")
try: urllib.URLopener().retrieve(url, filename)
except: urllib.request.urlretrieve(url, filename)
```

```python
# sample execution (requires torchvision)
# 실행예시 (torchvision이 요구됩니다.)
Copy link
Contributor

Choose a reason for hiding this comment

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

본문과 맞춰서 . 이 없는게 더 깔끔할 것 같습니다!

comparing to models with the same accuracy (except the two DS models that
were designed for comparing with MobileNet).
HardDNet(Harmonic DenseNet)은 낮은 메모리 트래픽을 가지는 CNN 모델로 빠르고 효율적입니다.
기본 개념은 계산 비용과 메모리 접근 비용을 최소화하는 것입니다. 따라서 HardDNet 모델은 동일한 정확도를 가진 ResNet 모델에 비해 GPU에서 실행되는 속도가 35% 더 빠릅니다. (MobileNet과 비교하기 위해 설계된 두 DS 모델은 제외)
Copy link
Contributor

Choose a reason for hiding this comment

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

The basic concept is to minimize both computational cost and memory access cost at the same
time,
기본 개념은 계산 비용과 메모리 접근 비용을 최소화하는 것입니다.

이 부분에 at the same time을 살려서 동시에 나 한번에 등의 단어를 추가 하는 것은 어떤가요!

기본 개념은 계산 비용과 메모리 접근 비용을 동시에 최소화하는 것입니다.

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
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.

@cpprhtn
오타 수정 부탁드립니다.

if torch.cuda.is_available():
input_batch = input_batch.to('cuda')
model.to('cuda')

with torch.no_grad():
output = model(input_batch)
# Tensor of shape 1000, with confidence scores over Imagenet's 1000 classes
# Imagnet의 1000개 클래스에 대한 신뢰도 점수를 가진 1000 형태의 텐서 출력
Copy link
Contributor

Choose a reason for hiding this comment

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

Imagnet을 ImageNet으로 바꿔야 할 것 같습니다!

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

@codingbowoo codingbowoo left a comment

Choose a reason for hiding this comment

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

오탈자 수정해주시면 머지하겠습니다. 수고하셨습니다!

pytorch_vision_hardnet.md Outdated Show resolved Hide resolved
pytorch_vision_hardnet.md Outdated Show resolved Hide resolved
cpprhtn and others added 2 commits September 14, 2022 23:44
Co-authored-by: 장보우 Bowoo Jang <lisabowoo@gmail.com>
Co-authored-by: 장보우 Bowoo Jang <lisabowoo@gmail.com>
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.

리뷰 반영 된 것 확인했습니다! 감사합니다.

@hyoyoung hyoyoung merged commit df9e463 into PyTorchKorea:master Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants