Feedback #1
Open
Feedback #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-classroom[bot]](https://avatars.githubusercontent.com/in/68187?s=60&v=4){kind=small}
Feat/ Wandb plus
Feat/ add albumentation augmentation
Feat/ optimizers
Feat/scheduler
실행 방식 추가 및 부제목 수정
학습 코드 및 추론 코드 실행 내용 위치 변경
gradcam.py 설명 관련 추가
mybirth0407
reviewed
Oct 9, 2024
There was a problem hiding this comment.
전체적으로 잘 작성된 코드이고, 흠잡을만한 부분이 없습니다. 다만, 일부 코드들은 주석이 부족하여 주석을 작성하면서 코딩하는 습관을 들이시면 더 좋을 것 같습니다. 고생한 흔적이 여기저기 많이 보입니다. 아마 코드를 주력으로 작성하신 분들과, 보조하신 분들로 나눠질 것 같은데요. 이 코드는 일부만 수정하면 분류, 객체 탐지 등 시각 이해에 대한 문제들은 모두 해결할 수 있는 코드입니다. 보조하신 분들도 이 코드를 꼭 전부 숙지하실 수 있도록 시간을 내어 공부하시면 반드시 도움이 되니, 꼭 공부하시길 바랍니다.
우리 16조 여러분 너무 너무 고생 많았습니다. 항상 화이팅입니다!
There was a problem hiding this comment.
- 리드미 내용 정말 좋습니다.
- 환경 및 버전에 PyTorch가 추가되면 좋겠습니다.
There was a problem hiding this comment.
- 더 다양한 스케쥴러와 옵티마이저를 지원하면 좋겠습니다.
- 상당히 훌륭합니다.
There was a problem hiding this comment.
- 색상 공간을 분석한 것은 좋은데, 이것이 실험의 아이디어로 이어졌으면 더 좋았을 것 같습니다.
There was a problem hiding this comment.
- torchvision과 albumentations를 혼용해서 쓰면 혼란의 우려가 있습니다. albumentations만 활용하는 것이 더 좋겠습니다.
- 문득 코드를 보다가 느끼는 것인데, 실시간 추론 성능이 중요하지 않은 대회라면, object detection 모형을 이용하여 객체의 위치를 대략 가늠한 후, 그 위치 내에서 mixup이나 cutmix를 도입해봐도 좋을 것 같습니다.
There was a problem hiding this comment.
- 53번째 행, img = self.transform(img)에서 transform이 없는 경우도 있을 수 있으니 검사하는 과정이 있으면 더 좋겠습니다.
There was a problem hiding this comment.
- CustomLoss로 클래스를 만든만큼, 더 다양한 손실 함수 구현이 필요합니다.
- CE의 인자로 클래스별 가중치를 넣어주는 옵션도 필요합니다.
- focal loss까지는 자주 활용되니 구현 연습해보시면 좋겠습니다.
There was a problem hiding this comment.
- 더 많은 스케쥴러 추가가 필요합니다.
- https://gaussian37.github.io/dl-pytorch-lr_scheduler/, https://timm.fast.ai/SGDR 와 같은 더 고급 스케쥴링 알고리즘도 알아두시면 좋습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
👋! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to the default branch since the assignment started. Your teacher can see this too.
Notes for teachers
Use this PR to leave feedback. Here are some tips:
For more information about this pull request, read “Leaving assignment feedback in GitHub”.
Subscribed: @hocheol0303 @Ja2Hw @hyanghyanging @sejongmin @whthdbs03 @kkh090