-
Notifications
You must be signed in to change notification settings - Fork 41
Fix: Replace AIML with Gen-AI chatbot integration #49
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
base: master
Are you sure you want to change the base?
Conversation
# License along with HablarConSara.activity. If not, see | ||
# <http://www.gnu.org/licenses/>. | ||
|
||
import time |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason in changing the license agreement statements at the start of the code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from gettext import gettext as _
from gi.repository import Gdk
from gi.repository import GLib
from gi.repository import Gio
from sugar3 import profile
from aiml.Kernel import Kernel
import voice
You have removed all the existing imports?
How do you think you will get profile information from about the user?
Can you please mention the specific files you are talking about? |
This particular file is brain.py |
|
|
Thanks James for bringing into my consideration. I am going to work on your suggestion and do the following amends. |
Summary
Refactors the chatbot in Speak Activity to use Gen-AI for more natural conversations.
Motivation
The AIML-based chatbot was limited. Gen-AI enables dynamic, context-aware responses and improves user experience.
Key Changes
ai_services.py
for modular Gen-AI logicHow to Test
OPENAI_API_KEY
in your environment.Feedback Requested
Thanks for taking the time to review this PR!