fix: improve grouped dormitory facility search#36
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a conditional search mechanism for dormitory-related queries, implementing a per-dormitory search strategy when specific keywords are detected in the user's question. The feedback focuses on cleaning up the implementation by removing debug print statements, externalizing hardcoded trigger keywords to constants or settings for better maintainability, and removing redundant hardcoded parameters that match default values in the search function.
| print("===== GROUPED SEARCH DEBUG =====") | ||
| print("question:", question) | ||
| print("chunks_count:", len(chunks)) | ||
| for index, chunk in enumerate(chunks, start=1): | ||
| print( | ||
| index, | ||
| chunk.get("document_id"), | ||
| chunk.get("dormitory"), | ||
| chunk.get("similarity"), | ||
| chunk.get("source"), | ||
| (chunk.get("content") or "")[:300], | ||
| ) | ||
| print("================================") |
There was a problem hiding this comment.
디버깅을 위해 추가된 print 문들이 코드에 남아 있습니다. 운영 환경의 로그를 오염시킬 수 있으므로, 이를 제거하거나 logging 모듈을 사용하여 적절한 로그 레벨(예: DEBUG)로 기록하는 것을 권장합니다.
| dormitory_specific_triggers = [ | ||
| "휴게실", | ||
| "다리미", | ||
| "편의점", | ||
| "전자레인지", | ||
| "전자렌지", | ||
| "정수기", | ||
| "세탁실", | ||
| "탕비실", | ||
| "수용인원", | ||
| "몇명", | ||
| "몇명수용", | ||
| "호실수", | ||
| ] |
There was a problem hiding this comment.
dormitory_specific_triggers 리스트가 함수 내부에 하드코딩되어 있습니다. 이러한 키워드 목록은 정책 변화에 따라 자주 수정될 수 있으므로, 유지보수 편의를 위해 모듈 수준의 상수로 분리하거나 Settings 클래스에서 관리하는 것이 좋습니다.
| dormitory_chunks = search_hybrid_chunks( | ||
| db=db, | ||
| query_text=question, | ||
| query_embedding=query_embedding, | ||
| dormitory=dormitory, | ||
| top_k=top_k_per_dormitory, | ||
| candidate_k=20, | ||
| keyword_weight=0.3, | ||
| ) |
There was a problem hiding this comment.
candidate_k와 keyword_weight 값이 하드코딩되어 있습니다. 이 값들은 search_hybrid_chunks 함수의 기본값과 동일하므로 생략 가능하며, 만약 명시적으로 관리하고 싶다면 Settings나 공통 상수를 사용하여 중복을 피하는 것이 좋습니다.
dormitory_chunks = search_hybrid_chunks(
db=db,
query_text=question,
query_embedding=query_embedding,
dormitory=dormitory,
top_k=top_k_per_dormitory,
)1 participant
유형
변경 사항
dormitory = null상태에서 생활관별 시설 질문을 처리할 때, 전체 검색 top_k 안에 특정 생활관 정보가 누락되는 문제를 개선했습니다.수정 배경
휴게실에 뭐 있어?처럼 생활관을 특정하지 않은 질문에서 제1, 제3학생생활관 정보는 검색되었지만 제2학생생활관 정보가 top_k 밖으로 밀려 답변에서 누락되는 문제가 있었습니다.테스트
휴게실에 뭐 있어?+dormitory = null다리미 있어?+dormitory = null기숙사 수용 인원 어느 정도야?+dormitory = null기대 효과