alexylem / jarvis

Jarvis.sh is a simple configurable multi-lang assistant.
http://openjarvis.com
MIT License
805 stars 197 forks source link

Réponse multiple Jarvis #500

Closed Lusitanos67 closed 7 years ago

Lusitanos67 commented 7 years ago

Je voulais savoir s'il était possible de définir une liste de réponses par défauts, par exemple après la détection du hotword, avoir soit oui/je t'écoute/c'est moi/ etc... Ça rendrait Jarvis plus convivial ?

alexylem commented 7 years ago

Aujourd'hui la réponse au hotword de Jarvis peut-être définie dans Settings / Phrases. Si je comprends bien, tu aimerais qu'il y ait la possibilité d'en définir plusieurs, donc Jarvis en dira une aléatoirement?

Lusitanos67 commented 7 years ago

Oui c'est ça

alexylem commented 7 years ago

Bon j'ai plusieurs idées dans la tête qui tournent autour de ça:

Approche simple Au lieu d'une boite de dialog "input", ouvrir un fichier (comme les hooks), une ligne par phrase possible.

Approche plus compliquée mais plus flexible/puissante Utiliser les hooks au lieu des phrases Pour ton exemple, dans entering_cmd on peut y mettre par défaut:

say "Oui?"

que tu pourrais facilement remplacer par:

answers=("Oui?" "C'est moi" "Je t'écoute" "Quoi encore?")
say "${answers[$RANDOM % ${#answers[@]} ]}"

Et on peut aussi utiliser program_startup pour:

say "Bonjour"

Et il faudrait donc rajouter les hooks suivants:

Avantages:

Inconvénients:

Traductions des phrases Ca fait un petit moment que je me dis qu'il faut que je traduise ces phrases. Par défaut elles sont en anglais et ca doit être un peu déroutant pour ceux qui installent Jarvis. Ca je peux le faire quelque-soit la solution choisie.

Lusitanos67 commented 7 years ago

Je teste la méthode complexe, elle me va :) Comme tu dis, pour les débutants, la compréhension risque de se faire difficilement, il faudrait rajouter de l'aide sur la partie "hook", ou repenser l'accès paramétrage de cette partie (ou paramétrage accessible en mode soit "simple" avec un assistant, ou en mode "expert").

Lusitanos67 commented 7 years ago

Ca fonctionne nickel