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

[2022 Spring] ICML 2020 SIMCLR (20225083) #505

Open
owojin-jeon opened this issue May 22, 2022 · 1 comment
Open

[2022 Spring] ICML 2020 SIMCLR (20225083) #505

owojin-jeon opened this issue May 22, 2022 · 1 comment

Comments

@owojin-jeon
Copy link

SimCLR이 self supervised learning 쪽에서 매우 유명한 논문이라고 알고있는데 이렇게 리뷰 작성 해 주셔서 감사합니다. 이해에 필요한 부분은 대부분 설명 해 주신 것 같아 좋았습니다. 조금 아쉬웠던 부분은 아무래도 논문을 영어로 읽다 보니 한글로 번역된 단어들이 익숙하지 않은데, 옆에 본문에서 사용된 용어도 같이 적어주셨으면 좋았을 것 같습니다. (ex: 사전 훈련, 자기주도학습 등). 그리고 TA 노트나 깨진 이미지와 같은 부분을 검토 해주셨으면 더 좋은 글이 되었을 것 같네요.

@owojin-jeon owojin-jeon changed the title [2022 Spring] ICML 2020 SIMCLR [2022 Spring] ICML 2020 SIMCLR (20225083) May 22, 2022
@drumpt
Copy link

drumpt commented May 22, 2022

잘 읽었습니다! SimCLR의 전반적인 내용이 적절히 설명되어 있는 것 같습니다. 몇 가지 피드백이 있는데요,

  • self-supervised learning, task 등의 용어를 한국어로 번역한 것이 오히려 어색한 것 같습니다.
  • Method paragraph의 바로 윗 부분에 불필요한 bullet이 있는 것 같습니다.
  • 자기'주도' 학습 이라는 용어를 쓴 것을 확인했는데 이것을 자기'지도'학습으로 바꾸는 것이 더 정확할까요?
  • 각각의 이미지를 잘 설명하는 적절한 캡션을 달아주시면 더 좋을 것 같습니다.
  • 'Data Augmentation for Contrastive Representation Learning' 단락의 이미지가 깨져 있는 것 같아 확인해주시면 감사하겠습니다.
  • 번역기를 쓴 것 같은 느낌이 들어서 이를 수정하면 좋을 것 같습니다. 예를 들어, 반말과 존댓말이 혼용되어 있는데, 글 전체를 존댓말로 통일하면 좋을 것 같습니다.
  • '본 논문은 아래와 같은 세 가지를 보였습니다.' 라는 문장이 있는데, 그 아래에 세 가지가 아니라 네 가지 사항이 적혀 있어 이를 '네 가지'로 수정하면 좋을 것 같습니다.
  • batch 내에서 loss term이 가지는 직관적인 의미(positive pair 사이의 similarity를 최대화 하는 것)를 서술하면 더 좋을 것 같습니다.

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

No branches or pull requests

2 participants