boostcampaitech5 / level1_imageclassification-cv-07

level1_imageclassification-cv-07 created by GitHub Classroom
1 stars 1 forks source link

[T5084] baseline v2 기반, 수정한 디렉토리 구조 #1

Closed nstalways closed 1 year ago

nstalways commented 1 year ago

금요일 피어 세션 때 얘기했던 내용대로 디렉토리를 만들고, 코드가 동작하도록 수정한 상태입니다.

추가적인 변경 사항이 생겼는데요. 원래 common 폴더에 있던 train.py, sample.submission.py, inference.pyroot_dir 아래로 이동 되었습니다.

common 폴더 아래에 있는 train.py를 터미널 상에서 실행시켰을 때 다른 폴더들을 import하지 못하는 에러가 발생해서, 이를 해결하기 위해 우선은 터미널 상에서 실행할지도 모르는 코드들을 root_dir 아래로 옮기게 되었습니다.

이외에 얘기하고 싶은 사항은

  1. train.py가 생각보다 변경 사항이 많을 수 있겠다는 점입니다. 예를 들어, pretrained 모델을 사용해서 학습을 진행한다고 하면 optimizer가 2개 필요하기 때문에 train 코드에 변경이 생기게 됩니다. 또 다른 구조로 진행한다면 train.py가 또 다르게 변경될 수 있을 것 같아요. 그래서 train.py를 어떤 방식으로 작성할 지 얘기해보았으면 좋겠습니다.

  2. model 폴더를 어떻게 할 지 baseline v2를 기반으로 학습 시킨 모델을 테스트로 올려보았는데요, 생각보다 용량을 많이 차지합니다. 그래서 모든 실험 결과를 공유하긴 힘들 것으로 보이고, 최고 성능을 보이는 모델만 업로드를 해야 할 듯 한데, 다들 생각이 어떠신지 궁금합니다.

codehyunn commented 1 year ago

2.에 대해서는 말씀해주신 대로 최고 성능을 내는 모델만 올리는 게 좋을 것 같네요. 처음에는 현재 실험한 모델 중에서 최선을 올리고, 더 나은 모델이 나오면 변경하는 식으로 진행하면 좋을 것 같습니다.

97DongHyeokOH commented 1 year ago
  1. 저도 train.py가 생각보다 변경 점이 꽤 생길 것 같아서 생각을 조금 더 해봐야 될 것 같습니다.
  2. 수영님, 지현님 의견과 같이 최고 성능 모델을 올리는 방식이 좋은 것 같습니다.
nstalways commented 1 year ago

의견 감사합니다. 코드 쪽에는 수정 사항이 없을까요? 수정 사항이 없으시다고 하면, merge를 해도 괜찮을까요?

codehyunn commented 1 year ago

네 merge해도 좋을 것 같습니다 ~

kjs2109 commented 1 year ago

저도 확인했습니다. 1, 2번은 저도 다른 분들과 같은 의견입니다. 추가적으로 제가 데이터 불균형 문제를 조금 해결해 보고 싶어서 여러가지 시도를 하고 있는데 dataset 코드를 조금 수정할 수도 있을 것 같아서 말씀드립니다. 아직 확실하지는 않지만 조금 코드를 추가할 수도 있을 것 같아요!

nstalways commented 1 year ago

다들 피드백 감사합니다! 그럼 merge하도록 하겠습니다. 주성 님께서도 코드 추가하실 때 branch에서 작업하시고, PR 올려주시면 리뷰하도록 하겠습니다!

kjs2109 commented 1 year ago

네 알겠습니다. 감사합니다!