Skip to content

AI03 - Improve Retrieval Flow for chat_with_rag#35

Open
Elly168 wants to merge 2 commits intoGopher-Industries:mainfrom
Elly168:feature/ai03-retrieval-flow
Open

AI03 - Improve Retrieval Flow for chat_with_rag#35
Elly168 wants to merge 2 commits intoGopher-Industries:mainfrom
Elly168:feature/ai03-retrieval-flow

Conversation

@Elly168
Copy link
Copy Markdown
Collaborator

@Elly168 Elly168 commented Mar 28, 2026

This PR improves the retrieval flow for chat_with_rag to enhance RAG response quality.

Changes made

  • Improved retrieval flow to return more relevant context
  • Reduced duplicate and low-quality retrieved chunks
  • Made retrieval results more consistent for user queries
  • Added notebook to test and validate retrieval improvements

Purpose

This supports AI03 by improving how context is retrieved before generating responses, leading to better and more accurate outputs.

@Elly168 Elly168 requested a review from TienNguyen3711 March 28, 2026 11:55
@Elly168 Elly168 assigned Elly168 and unassigned Elly168 Mar 28, 2026
@Elly168 Elly168 requested review from Liam301020 and removed request for TienNguyen3711 March 28, 2026 12:04
Copy link
Copy Markdown
Collaborator

@Liam301020 Liam301020 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally. The notebook-based retrieval validation works as expected and the sample outputs look more relevant. Based on the current diff, this PR appears to demonstrate retrieval testing/validation rather than a direct runtime update to /ai-model/chatbot/chat_with_rag.

@Liam301020
Copy link
Copy Markdown
Collaborator

The current notebook changes are useful for validation, but AI03 requires improvement in the active RAG service / retrieval flow itself. Based on the current diff, additional runtime retrieval changes may still be needed to fully satisfy the task scope and DoD.

@Elly168 Elly168 closed this Apr 3, 2026
@Elly168 Elly168 reopened this Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants