Skip to content
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

Configurable persona #472

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Azelphur
Copy link

This adds support for a configurable persona (trigger word)

You can set persona in the config file, by default JASPER is assumed.

examples:

persona: JASPER
persona: COMPUTER
persona: JARVIS

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 41.212% when pulling 3ef1b1e on Azelphur:configurable_trigger into e6789d7 on jasperproject:master.

WORKs

Choose a reason for hiding this comment

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

this extra s looks like a missed ctrl+s ?

@Azelphur
Copy link
Author

Whoops! sorted now. :)

So many typos... this is what I get for coding on my laptop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 41.47% when pulling ec02fbe on Azelphur:configurable_trigger into e6789d7 on jasperproject:master.

@ThomasDelteil
Copy link

no worries, FYI I noticed that on the jasper-dev branch, the configurable persona feature has already been implemented, using the keyword keyword in the profile.yaml file. Though I am not familiar with the current roadmap and when it is planned to merge jasper-dev in master.

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