boostcampaitech7 / level2-mrc-nlp-06

level2-mrc-nlp-06 created by GitHub Classroom
3 stars 1 forks source link

[BUGFIX] Sparse Retrieval 경로 문제 해결 #20

Closed minjijeong98 closed 1 week ago

minjijeong98 commented 1 week ago

Overview

Change Log

예시 - 변경 후

sparse_path_name = os.path.dirname(os.path.dirname(os.path.abspath(file))) pickle_name = f"{self.embedding_method}_sparse_embedding.bin"



## To Reviewer
- BUG 이슈에서 제안해주신 config에 경로를 추가하는 방식도 고민했지만, 다음과 같은 두 가지 이유로 해당 방식을 채택했습니다.    
    - 첫째, 팀원마다 서버에 이 프로젝트 파일이 저장된 경로들이 약간씩 다름. 
        - 예: `/data/ephemeral/home/minji/retrieval/sparse/model/`에서 `minji` 등
        - 매 번 각자의 환경에 맞춰 config에서 파일 경로 수정하기 번거로울 것 
    - 둘째, `get_sparse_embedding` 함수들 뿐만 아니라 다른 경로들에도 공통적으로 수정이 필요해 보임
        - `run_sparse_retrieval.py` 및 `utils_sparse_retrieval.py` 등
- @LeeJeongHwi 확인부탁드려요:) 별다른 이상 없다고 하시면 바로 병합하겠습니다.

## Issue Tags
- Fixed: #18 
LeeJeongHwi commented 1 week ago

이상없습니다! 감사합니다 :) 더 좋은 방법이네요!