search

boostcampaitech3 / level2-data-annotation_cv-level2-cv-09

level2-data-annotation_cv-level2-cv-09 created by GitHub Classroom
0 stars 4 forks source link

[Discussion] git issue format 정하기 #4

Closed jeongjae96 closed 2 years ago

jeongjae96 commented 2 years ago

git issue format도 convention을 정하면 좋을 것 같습니다!

더 많은 issue convention이 필요할듯 한데 여기서 의논해보고 싶습니다!

04/12 : Feat, EDA 사용하기로 결정 04/14 : Test, Data, Refactor, Style 추가

yoonghee commented 2 years ago

저는

  1. [Feat]
  2. [EDA] 로 의미가 다 전달되기도 하고 간결해서 제목까지 보는데 더 좋을 것 같습니다~
jeongjae96 commented 2 years ago

Feat, EDA 사용하는 것으로 업데이트했습니다!

jeongjae96 commented 2 years ago

[Test] : test 용도의 issue 생성할 때 사용할까요?

jeongjae96 commented 2 years ago

이슈 포맷에 [Refactor]를 추가할까 하는데, 의견 부탁드립니다.

현재 convert_mlt.py line 18 부분의 코드는 다음과 같습니다.

IMAGE_EXTENSIONS = {'.gif', '.jpg', '.png'}

'.jpeg' 확장자도 받기 위해 코드를 다음과 같이 수정하려고 합니다.

IMAGE_EXTENSIONS = {'.gif', '.jpg', '.png', '.jpeg'}

ICDAR 데이터셋에는 기존 확장자만 사용하여도 충분하지만, 추후 다른 확장자가 들어간 데이터 변환 시 추가해도 될 것 같습니다.

이런 이유로 issue를 새로 생성하고 작업하려고 하는데, [Refactor]로 열면 어떨까요?

yoonghee commented 2 years ago

저도 지금 train.py model.py 수정하고 있는 부분이 있어서 [Refactor]가 생기면 좋을것 같네요~

km9mn commented 2 years ago

@yoonghee model.py는 수정하면 안되지 않나요?

yoonghee commented 2 years ago

@yoonghee model.py는 수정하면 안되지 않나요?

저도 그렇게 들었던것 같긴한데 아까 epoch 끊긴부분 loss 값 받는 부분만 예외처리해서 잘 동작하는지 확인해보려구요

jeongjae96 commented 2 years ago

[Style]도 있으면 좋을 것 같아 추가합니다~