Skip to content
This repository was archived by the owner on Sep 1, 2025. It is now read-only.

Conversation

@jingai
Copy link
Collaborator

@jingai jingai commented Aug 26, 2017

This expands the Help response to include random example utterances the user can try.

The spoken response includes one random utterance and another for the reprompt. The same reprompt is used for ask.launch (skill opened with no Intent requested, as in, open kodi).

The Card presented in the Alexa app provides 5 randomly-chosen utterances.

The example utterances are chosen from the speech_assets/SampleUtterances.LANG.txt file. A new file sample_slotvals.LANG.txt was created to fill slot references with something to make it sound more natural than something like insert title here.

Needs testing for Heroku deployments

@jingai jingai requested a review from m0ngr31 August 26, 2017 17:57
@jingai
Copy link
Collaborator Author

jingai commented Aug 26, 2017

There are several things in this PR that require German translations:

  • sample_slotvals.de.txt -- the media titles don't necessarily need to differ, but things like thirty seconds do need localization,
  • optional: Extensions to sample utterances for AMAZON.HelpIntent in utterances.de.txt,
  • The new/modified template strings:
help: "You can ask me whether there are any new shows, to play a movie, t. v. show, or artist, control playback of media, and more. For example, {{ example }}. I've sent more examples to your Alexa app."

help_short: "What would you like to do?  Try something like, {{ example }}."

help_text: "Kodi-Alexa allows you to fully control one or more Kodi installations with your voice.  Here are some examples of things you can say:\n\n{{ examples }}\nFor a complete list of supported requests, please see the README file on Github (https://github.com/m0ngr31/kodi-alexa/README.md)."

Ping @mcl22 and @ausweider :)

@jingai
Copy link
Collaborator Author

jingai commented Aug 26, 2017

@mcl22 and @ausweider, I forgot to mention.. the values for these in sample_slotvals.de.txt should all be related:

Artist
Album
MusicGenre
Song
MusicVideoGenre
MusicVideo

That is, they should ideally all be from the same album and by the same artist, otherwise you could get some weird suggestions like:

Play Death Metal music by The Beetles
Play Barbie Girl by Metallica

;)

@mcl22
Copy link

mcl22 commented Aug 27, 2017

Ok, here we go. I hope I didn't miss anything :)

template strings:
help: "Du kannst mich fragen, ob es neue Serien gibt, mich bitten einen Film, eine Serie oder die Musik eines Interpreten abzuspielen oder die Wiedergabe von Medien zu steuern und vieles mehr. Zum Beispiel {{ example }}. Ich habe dir weitere Beispiele an deine Alexa App geschickt."
help_short: "Was möchtest du tun? Probiere etwas wie {{ example }}.""
help_text: "Kodi-Alexa ermöglicht es dir eine oder mehrere Kodi Installtionen mit deiner Stimme zu steuern. Hier sind einige Beispiele, die du sagen kannst:\n\n{{ example }}\nFür eine komplette Liste der unterstützten Kommandos, lese bitte die README Datei auf Github (https://github.com/m0ngr31/kodi-alexa/README.md)."

utterances:
AMAZON.HelpIntent (/nach) was ich es fragen kann
AMAZON.HelpIntent was es (/alles) kann

sample slotvals:
Addon YouTube
MovieGenre Komödie
Movie Männertag
Artist Helene Fischer
Album Farbenspiel
MusicGenre Schlager
AudioPlaylist Partymusik
Song Atemlos
Show Farscape
VideoPlaylist HD-Videos
ShowGenre Science-Fiction
MusicVideoGenre Pop
MusicVideo Die Hölle morgen früh
Volume Fünfundsiebzig
Season zwei
Episode acht
ForwardDur eine Minute
BackwardDur 30 Sekunden

@jingai
Copy link
Collaborator Author

jingai commented Aug 27, 2017

Thanks again @mcl22 :)

@digiltd
Copy link
Contributor

digiltd commented Aug 28, 2017

Just a thought, but given the double meaning of the word "alternative" is it going to be a little confusing when used in context?

ListenToGenre listen to alternative genre
ListenToGenre listen to alternative music
ListenToGenre listen to alternative songs

ListenToArtist listen to alternative artist Sia
ListenToArtist listen to alternative music by Sia
ListenToArtist play alternative artist Sia
ListenToArtist play alternative music by Sia

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

It doesn't sound weird to me, but honestly it doesn't matter to me either what it is so long as it fits, so I'll change it :)

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Have you tried this PR with AWS? If no one gets around to it, I can do it, but it'd save me some trouble.

@digiltd
Copy link
Contributor

digiltd commented Aug 28, 2017

It works.

image

Could quotes be added around each command to help with formatting?

I'm all for providing the user with help, but the randomness of the examples might not be that helpful, especially for a beginner.

Thoughts on using a curated list of the basics to randomly pick from?

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Yes, I can add quotes around them. I actually hadn't looked at it using the mobile app.

As for the list of things, I prefer a random set because it shows variations. Certain intents can be excluded, however. I've already excluded the PlayerZoom intents and had considered excluding the navigation commands because things like "right" without any context are kind of confusing, particularly in the spoken response.

edit: there should be newlines there too.. Amazon says they can be included, and they show up right in a web browser.. not sure why they're not there on mobile.

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Had to insert a non-breaking space in between two newlines to get it to show an empty line on the mobile app.. kinda weird, but it works. Lemme know if that looks better.

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

As an aside, it'd be nice if Amazon allowed hypertext links, though I can understand why they don't (security risk).

@digiltd
Copy link
Contributor

digiltd commented Aug 28, 2017

image

Peachy.

Maybe just say "for a complete list please see the GitHub repo"

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Thoughts on including the navigation/system commands? Particularly these:

Home (/navigate/go) home
Back (/navigate/go) back
Up (/navigate/go) up
Down (/navigate/go) down
Right (/navigate/go) right
Left (/navigate/go) left
PageUp (/navigate/go) page up
PageDown (/navigate/go) page down
PageDown show more
Select select
Menu (/open/context) menu
Fullscreen (/toggle) (fullscreen/visualizer)
Hibernate (hibernate/deep sleep) (/system)
Reboot reboot (/system)
Shutdown shut down (/system)
Suspend (suspend/sleep) (/system)
Suspend put system to sleep
Suspend go to sleep

@m0ngr31
Copy link
Owner

m0ngr31 commented Aug 28, 2017

That's fine with me

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Meaning you'd prefer to include or exclude them?

@m0ngr31
Copy link
Owner

m0ngr31 commented Aug 28, 2017

Sorry, I think it's fine to include those.

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Ok, all that needs to be done I think then is to test on Heroku.

@m0ngr31
Copy link
Owner

m0ngr31 commented Aug 28, 2017

I'll give it a test real quick

@m0ngr31
Copy link
Owner

m0ngr31 commented Aug 28, 2017

Works great on Heroku

@jingai
Copy link
Collaborator Author

jingai commented Aug 28, 2017

Awesome, thanks for checking.

Thanks to @mcl22 for the German translations.
@jingai jingai merged commit d1852e5 into m0ngr31:master Aug 28, 2017
@jingai jingai deleted the expanded-help branch August 28, 2017 23:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants