Closed minjijeong98 closed 2 weeks ago
아직 실행은 안 해봤지만 코드 내용만 확인했습니다!
전체적으로 깔끔하게 구분하여 구성하신 것 같습니다. command line arguments가 많지 않은 것도 좋습니다.
다만 sparse_retrieval/log
에 있는 log 파일들은 꼭 필요한 정보가 있는 게 아니라면 .gitignore에 포함하는 방식 등을 사용하여 트래킹하지 않아도 괜찮을 것 같습니다!
전체적으로 코드가 깔끔하게 잘 작성되어서 좋은 것 같습니다. inference를 구성하는 과정에서 기존 코드와 동일하게 작동해서, 작업하기 편할 것 같네요!
코드가 깔끔하게 되어 있어서 보기 편해 좋은 것 같습니다.
리뷰 감사합니다!! 그럼 병합하겠습니다:)
Overview
Folder Structure
retrieval/sparse
폴더는 다음과 같은 구조로 구성되었습니다.실행 방법
./src/sparse_retrieval.py
의SparseRetrieval
클래스를 활용합니다.run_sparse_retrieval.py
파일을 통해 수행 가능합니다.run_sparse_retrieval.py
과 config 파일명을 입력해 실행합니다.To Reviewer
sparse_retrieval
폴더를 남겨두었습니다.11 이슈 해결하는대로 learned sparse retrieval 구현 마무리하면서 해당 폴더는 삭제할 예정입니다.
Issue Tags