Skip to content

feat: wire Qdrant retrieval into LLM pipeline & improve embedding worker#129

Merged
mhd-hi merged 8 commits into
devfrom
query-time-results-to-llm-pipeline
Jun 22, 2026
Merged

feat: wire Qdrant retrieval into LLM pipeline & improve embedding worker#129
mhd-hi merged 8 commits into
devfrom
query-time-results-to-llm-pipeline

Conversation

@mhd-hi

@mhd-hi mhd-hi commented Jun 15, 2026

Copy link
Copy Markdown
Member

⁉️ Related Issue

closes #119

📖 Description

  • Connected CourseRetrialService to LlmService, chatbot/recommend now gets the context of the "query-time" (top-k) courses.
  • Changed bge-m3 download from mean to cls, seems to be more reliable, also added progress to observe bge-m3 model downloading.

🧪 How Has This Been Tested?

  • ad-hoc /chatbot/recommend tested
  • write tests

☑️ Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

🖼️ Screenshots (if useful)

@mhd-hi mhd-hi mentioned this pull request Jun 15, 2026
2 tasks
@github-actions

Copy link
Copy Markdown

Test Coverage

Lines Statements Branches Functions
Coverage: 88%
88.9% (5267/5924) 91.72% (865/943) 92.39% (316/342)

@mhd-hi mhd-hi requested a review from Jung500 June 22, 2026 22:44

@Jung500 Jung500 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@mhd-hi mhd-hi merged commit 82d3905 into dev Jun 22, 2026
2 checks passed
@mhd-hi mhd-hi deleted the query-time-results-to-llm-pipeline branch June 22, 2026 22:54
@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants