[CHORE] 아이디어 생성 시, source_id str으로 수정 => source_id str에서 int 타입 변경 #167
Conversation
Summary of ChangesHello @Ochangmin524, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 아이디어 생성 과정에서 사용되는 Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
external/rag/rag_service_impl.py
Outdated
|
|
||
| video_embedding = await self.content_chunk_repository.generate_embedding(query_text) | ||
| meta_data = {"query_embedding": str(video_embedding), "source_id": int(channel.channel_hash_tag.value)} | ||
| meta_data = {"query_embedding": str(video_embedding), "source_id": str(channel.channel_hash_tag.value)} |
There was a problem hiding this comment.
source_id를 문자열로 변환하는 것은 올바른 방향입니다. 하지만 get_popular_videos 메서드(129행)에서는 content_chunk를 저장할 때 source_id를 int로 저장하고 있습니다:
# external/rag/rag_service_impl.py:129
source_id=int(category.value)데이터를 저장할 때는 정수형으로, 검색할 때는 문자열로 source_id를 사용하면 search_similar_by_embedding이 의도한 대로 동작하지 않고 유사한 청크를 찾지 못할 가능성이 매우 높습니다. 데이터 저장과 검색 시 source_id의 타입을 일관되게 문자열로 사용하도록 get_popular_videos 메서드도 함께 수정해야 합니다.
🤖 Gemini AI 코드 리뷰❌ Gemini API 호출 실패: 최대 재시도 횟수 초과 이 리뷰는 Gemini AI가 자동으로 생성했습니다. 참고용으로만 활용해주세요. |
* /ideas * consumer v2 버전 테스트 완료
🤖 Gemini AI 코드 리뷰❌ Gemini API 호출 실패: 최대 재시도 횟수 초과 이 리뷰는 Gemini AI가 자동으로 생성했습니다. 참고용으로만 활용해주세요. |
yujin9907
changed the title
PR 제목
[Feat/Fix/Refactor/Docs/Chore 등]: 간결하게 변경 내용을 요약해주세요. (예: Feat: 사용자 로그인 기능 구현)
✨ 변경 유형 (하나 이상 선택)
📚 변경 내용
구체적으로 어떤 변경 사항이 있는지, 왜 이러한 변경이 필요한지 설명해주세요.
(예: 사용자 회원가입 시 이메일 중복 확인 로직 추가. 기존 로직에서 누락된 부분 발견하여 수정.)
⚙️ 주요 작업 (선택 사항)
✅ 체크리스트
🔗 관련 이슈 (선택 사항)
해당 PR이 해결하는 이슈 또는 관련 있는 이슈가 있다면 링크를 걸어주세요.
(예: #123, ABC-456)
💬 기타 (선택 사항)
리뷰어에게 전달하고 싶은 추가 정보나 궁금한 점이 있다면 작성해주세요.