-
Notifications
You must be signed in to change notification settings - Fork 16
GAN ch.1 update after some corrections from issue tickets #57
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
pr from lab's repo to my repo
| "GAN은 구글 브레인에서 머신러닝을 연구했던 Ian Goodfellow에 의해 2014년 처음으로 신경정보처리시스템학회(NIPS)에서 제안되었고, 이후 텍스트 생성, 이미지 생성 등에 다양하게 응용되고 있습니다." | ||
| "GAN은 구글 브레인에서 머신러닝을 연구했던 Ian Goodfellow에 의해 2014년 처음으로 신경정보처리시스템학회(NIPS)에서 제안되었고, 이후 텍스트 생성, 이미지 생성 등에 다양하게 응용되고 있습니다.\n", | ||
| "\n", | ||
| "이번 1장에서는 GAN의 개념을 비롯하여 GAN 모델의 구조와 평가지표, 적용 사례 등에 대해 알아봅니다. 1.1절에서는 GAN의 등장과 개념에 대해서 설명하고 1.2절에서는 GAN 모델을 구조와 GAN의 한 종류인 CGAN 모델, 그리고 GAN 모델의 평가지표에 대해서 설명합니다. 이어 1.3절에서는 GAN을 적용한 사례들을 살펴보며 1.4절에서는 GAN이 가진 한계점을 설명하고 있습니다." |
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.
1.2절에서는 GAN 모델을 구조와 GAN의 한 종류인 CGAN 모델
->
1.2절에서는 GAN 모델의 구조와 GAN의 한 종류인 Conditional GAN (CGAN) 모델
이렇게 변경하는 것이 어떨까요? CGAN이 처음 나오는 부분이라서 약어 의미를 모를 수가 있을것 같습니다.
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.
넵 피드백 감사합니다. 해당 부분 반영하여 로컬&개인레포(포크된 레포와 다른 레포)에서는 Overview까지 어색한 문맥 수정을 해두었는데요, 아래 링크에서 참고하실 수 있을 것 같습니다. 1.1절부터 시작하여 어색한 문맥 수정 등 일단락이 되면 다시 PR해두겠습니다.
https://github.com/hwaneest/pseudo_lab_tutorial_book/blob/main/Ch1-Introduction.ipynb
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.
혹시 이 PR은 어떻게 되는 걸까요?! 취소하셔도 좋습니다. (현재 이렇게 PR이 대기중인 상태에서는 리모트에서 포크한 제 레포로 동기화하는 작업이 안 되는 것 같던데 맞나요...? 아니면 한 사람이 머지되지 않은 PR이 있어도 다른 PR을 동시에 추가로 해둘 수 있는 건가요?)
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.
hwaneest/Tutorial-Book repo에 수정사항 commit해두시면 이 PR도 업데이트가 됩니다
hwaneest/pseudo_lab_tutorial_book에 있는 수정사항을 hwaneest/Tutorial-Book에 반영해주시면 해결 될 것 같습니다.
작업을 바로 hwaneest/Tutorial-Book에 하지 않고 hwaneest/pseudo_lab_tutorial_book을 사용하시는 이유가 있을까요?
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.
그렇군요, 설명 감사합니다 :) 👍
따로 pseudo_lab_tutorial_book 레포를 사용해야만 하는 이유가 있는 것은 아닙니다. 다른 파일들을 생성해서 이리저리 테스트해보고 하는 용도로 hwaneest/Tutorial-Book을 깔끔하게 관리하기 위한 서포트 용도로 사용하고 있어요. 이슈 있을까요?
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.
repo 2개 관리하는게 불편하실 수도 있을 것 같아 여쭤봤습니다 ㅎㅎ 이리저리 테스트 하는 용도로 일반적으로 1개의 repo에 여러개의 branch를 생성해서 관리하는데, 저도 아직 익숙하지는 않은 방법이라... ㅎㅎ
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.
오호 그렇군요, 성진님 통해서 정말 많이 배웁니다. 그렇게 해봐야겠네요, 감사합니다.
(p.s. gitlocalize는 일단 submit해두고 있는데 그렇게 하는 게 맞는 것인지는(생각보다 간단하네요!); 이따 모임에서 마저 얘기나누어요!)
|
@SDSTony 성진님, 우선 제 쪽에서 1차적인 편집과 수정은 모두 끝이 났습니다. 향후 피드백이 추가로 있다면 반영하면 될 것 같고, PR 한 번 확인 부탁드립니다. 보시고 추가 issue 없겠다 싶으면 issue 클로즈해주세요. (PR에는 이미지 파일명과 디렉 수정도 포함되어 있고 현재는 PR 전이라 이미지는 설명과 매치되지 않아 보일 수 있습니다.) |
GAN ch.1 update after some corrections from issue tickets