OneBusAway / onebusaway-alexa

An Java-based app to communicate with Amazon Alexa for devices such as the Amazon Echo
Other
52 stars 18 forks source link

Address certification feedback #63

Closed barbeau closed 8 years ago

barbeau commented 8 years ago

Below is feedback from the Alexa certification team that we need to address before public launch (which requires another submission to certification with the below items resolved). I opened a WIP PR at https://github.com/OneBusAway/onebusaway-alexa/pull/64 with individual commits to address the below items, and will check them off as they are addressed.


Requirements:

rogue valley york york region tampa bay puget sound san diego m. t. s.

Additionally, consider adding any other city/area name variation that you think users might say. For more information on expanding built-in slot types, please visit: https://developer.amazon.com/public/solutions/alexa/alexa-skills-kit/docs/alexa-skills-kit-interaction-model-reference#Extending%20a%20Built-in%20Slot%20Type%20with%20Custom%20Values

Recommendations:

SetStopNumberIntent {stopNumber}

  • [x] Errors in the tokenization of acronyms and initialisms can greatly impact the speech recognition for your skill. All instances of "id" should be changed to "i. d.".
  • [x] The skill will work best for users when you include utterances that can easily work on their own, as well as utterances that work well to follow the one-shot launch phrasing. Consider adding the following types of sample utterances:

Commands (Work well after "ask onebusaway to..." or "tell onebusaway to...")

GetCityIntent get my current city GetCityIntent tell me my current city GetStopNumberIntent tell me what my stop is GetArrivalsIntent get me arrival times etc.

Noun Phrases (Work well after "ask onebusaway for...")

GetCityIntent my current region GetCityIntent my city GetStopNumberIntent my current stop GetStopNumberIntent my stop GetArrivalsIntent the arrival times of my {TransitMode} GetArrivalsIntent {TransitMode} times etc.

Interrogative and Inverted Questions (Work well after "ask onebusaway...")

GetArrivalsIntent where's my {TransitMode} GetArrivalsIntent where {TransitMode} is GetStopNumberIntent what's my current stop GetStopNumberIntent what my stop is GetCityIntent what's my city GetCityIntent what my city is etc.

barbeau commented 8 years ago

Alright, most of these have been pretty easy. Only one left related to adding more utterances. We're going to have to be careful with this one due to https://github.com/OneBusAway/onebusaway-alexa/issues/12 - I already had to remove the following two utterances in https://github.com/OneBusAway/onebusaway-alexa/commit/021279e27e1681998d3b1fa682832578b2aa53fc that were preventing it from correctly hearing "Atlanta":

GetArrivalsIntent the arrivals
GetArrivalsIntent for arrivals

So I want to test the recommended utterances to make sure we're not giving up recognition of any cities.

barbeau commented 8 years ago

Alright, just knocked off the last comment re: utterances. I'm now going to test and see if everything still works correctly.