Skip to content

feat: add mock LLM chatbot response function #38

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.

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

YedianCheng
Copy link

Summary

This PR introduces a mock version of the chatbot response function get_llm_response() to simulate LLM replies. It is part of my preparation for Google Summer of Code 2024 (GSoC) with Sugar Labs, under the proposal to enhance the Speak Activity with gen-AI capabilities.

What it does

  • Adds a mock LLM response function returning randomized phrases
  • Prepares the integration point for future LLM interaction and persona expression
  • Ensures compatibility with current UI and facial SVG expression features

Note: This is an early mock implementation for testing purposes only. The actual LLM logic will be implemented in a separate file (similar to brain.py) in future development.

Testing

  • The activity launches successfully
  • Chatbot responds with expected mock responses
  • No existing functionality is broken

Looking forward to feedback and happy to iterate further!

@YedianCheng YedianCheng changed the title Feat/mock llm response feat: add mock LLM chatbot response function Mar 28, 2025
@quozl
Copy link
Contributor

quozl commented Mar 28, 2025

Thanks. Could you convert to draft?

@YedianCheng YedianCheng marked this pull request as draft March 28, 2025 23:50
@YedianCheng
Copy link
Author

Thanks. Could you convert to draft?

Sure! I've converted it to a draft. Thanks for the suggestion.

@@ -1,5 +1,5 @@
[Activity]
name = Speak
name = Edie_Speak
Copy link
Member

Choose a reason for hiding this comment

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

Why the name and bundle_id change?

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