Skip to content

Update LLM-based NLG and NLU #218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
NoB0 merged 3 commits into from
Nov 18, 2025
Merged

Update LLM-based NLG and NLU #218

NoB0 merged 3 commits into from
Nov 18, 2025

Conversation

@NoB0
Copy link
Collaborator

@NoB0 NoB0 commented Oct 27, 2025 •
edited
Loading

  • Make llm_interfaces a higher level package
  • Refactor LLM-based NLU and NLG components to use LLMInterface instead of Ollama
  • Add tests for LLM-based NLU and NLG components
  • Update configuration structure

Fixes #213

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

Current Branch Main Branch
Coverage Badge Coverage Badge

@NoB0 NoB0 marked this pull request as ready for review October 27, 2025 13:16
@NoB0 NoB0 requested a review from kbalog October 27, 2025 13:16
@NoB0 NoB0 added the P0 Top priority label Oct 27, 2025
kbalog
kbalog approved these changes Nov 14, 2025
View reviewed changes
Copy link
Contributor

@kbalog kbalog left a comment

Choose a reason for hiding this comment

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

LGTM.

@NoB0 NoB0 merged commit 063177f into main Nov 18, 2025
4 of 5 checks passed
@NoB0 NoB0 deleted the feature/213-Update-LLM-based-NLG-and-NLU branch November 18, 2025 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kbalog kbalog kbalog approved these changes

Assignees

No one assigned

Labels

P0 Top priority

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update LLM-based NLG and NLU

3 participants

@NoB0 @kbalog