search

pmz-q / est_wassup_03

MIT License
1 stars 0 forks source link

Bugfix: detect_face.py #50

Open pmz-q opened 7 months ago

pmz-q commented 7 months ago
pmz-q commented 7 months ago

https://github.com/pmz-q/est_wassup_03/pull/46#issuecomment-1958501477

이거 참고해주세요

angie0bb commented 7 months ago

@pmz-q 아하 이거 그럼 둘다 작업된거죠?

pmz-q commented 7 months ago

@angie0bb 음, 저거는 그냥 image 파일 리스트 보고, coco annotation 엔 있는데 없는 이미지들의 경우, coco annotation 에서 제거해주는 로직이에요.

사실 이 이슈에서 어떤걸 하고자 하신건지 이해가 잘 안되어서 둘 다 해당하는 것인진 잘 모르겠슴다

angie0bb commented 7 months ago

아하 네 설명드리면,

  1. error case 저장되는 json 파일 경로 잘못들어가는 것 수정 
    with open(os.path.join(save_cropped_dir, "failed_detect_files.json"), "w", encoding="cp949") as json_file: # save error case
    json.dump(failed_detect_files, json_file)

    이 코드에 ) 오타들어간 부분이었고, 선들님이 다른 작업에서 함께 수정해주신 것 확인했구요.

  2. annotations.json 파일 copy하는 로직 추가
    • 이건 detect_face.py로 crop이미지 받아올떄 labels폴더는 따로 복사해오지 않아서 추가해야하는 부분이었습니다.
    • 이 부분만 추가하겠습니다!
pmz-q commented 7 months ago

이해했습니다!