Skip to content
This repository has been archived by the owner on Jan 16, 2024. It is now read-only.

Voice response is incomplete #839

Closed
zeusshuang opened this issue Jul 10, 2018 · 16 comments
Closed

Voice response is incomplete #839

zeusshuang opened this issue Jul 10, 2018 · 16 comments

Comments

@zeusshuang
Copy link

Hi AVS team,

SDK 1.8 , ARM based OpenWRT embedded linux.

We got many reported that the voice response is incomplete or cut off.
Sometime it happens while barge-in during playing Music, sometimes it's from normal Q&A case.
Here I attached a log from barge-in case.
While playing music, say: Alexa, Thumbs down. It response "okay", but from APP History, it should respond "okay rating saved".

19541-cc1d849774154846b9c75a02703bb28d.txt

@kclchan
Copy link
Contributor

kclchan commented Jul 11, 2018

Hi @zeusshuang,

Would you be able to point out the line number in the .txt file where the voice response is cut off?

Thanks,
Kenneth

@zeusshuang
Copy link
Author

Hi @kclchan ,

By tester, the timestamp is about 2018-07-09 03:42, where you will see a "KeywordObserver" log, it's barge-in, the response cut off should happens after this barge-in

@kclchan
Copy link
Contributor

kclchan commented Jul 11, 2018

Hi @zeusshuang,

Thanks for providing the timestamp, it's extremely helpful. I am looking at the logs now, and will get back to you ASAP once we find anything.

Thanks,
Kenneth

@kclchan
Copy link
Contributor

kclchan commented Jul 12, 2018

Hi @zeusshuang,

Based on the logs:

018-07-09 03:42:35.665 [ b] 0 DialogUXStateAggregator:transitionFromThinkingTimedOut
2018-07-09 03:42:35.798 [ 3] 0 MediaPlayer:callingOnTags
2018-07-09 03:42:35.798 [ b] 0 DialogUXStateAggregator:setState:from=THINKING,to=IDLE
2018-07-09 03:42:35.799 [ 3] 0 MediaPlayer:callingOnPlaybackStarted:currentId=260
2018-07-09 03:42:35.799 [ 3] C SpeechSynthesizer:onPlaybackStarted:METRICS::Location=SpeechSynthesizer Receive,NAME=SpeechStarted

The speech is cut off due to getting transitionFromThinkingTimedOut before the SpeechSynthesizer start speaking. There appears to be a race condition and we have created an internal ticket to track and fix this issue ASAP.

Thanks again for letting know about this issue and providing the logs for it.

Thanks,
Kenneth

@zeusshuang
Copy link
Author

Many thanks for your comments !

@jpatu001
Copy link

jpatu001 commented Sep 7, 2018

Hi All,

Is this issue resolved? I may have a similar issue. When cancelling a timer/ alarm, the response I get is "Timer for 10..." then gets cut off instead of "Timer for 10 seconds cancelled". My logs show that the transition I have when cancelling is Listening->Thinking->Idle->Speaking->Idle. This is on sdk v1.8.1 embedded linux..

Thanks,
jpatu001

@jan4984
Copy link

jan4984 commented Sep 20, 2018

yes I have this problem in some arm Linux board . is this a issue of gstreamer? https://lists.freedesktop.org/archives/gstreamer-devel/2015-March/052131.html

please any one know gstreamer can help fix this for avs-device-sdk?

@jan4984
Copy link

jan4984 commented Sep 21, 2018

I found that the version 1.5 sdk 's sleep works in 1.7.1. not know why amazon remove it.

MediaPlayer.cpp:

case GST_MESSAGE_EOS:
    std::this_thread::gp_for(std::chrono::milliseconds{500});

scotthea-amazon added a commit that referenced this issue Oct 24, 2018
Changes in this update:

**Enhancements**

* New optional configuration for [EqualizerController](https://github.com/alexa/avs-device-sdk/blob/v1.10.0/Integration/AlexaClientSDKConfig.json#L154). The EqualizerController interface allows you to adjust equalizer settings on your product, such as decibel (dB) levels and modes.
* Added reference implementation of the EqualizerController for GStreamer-based (MacOS, Linux, and Raspberry Pi) and OpenSL ES-based (Android) MediaPlayers. Note: In order to use with Android, it must support OpenSL ES.
* New optional configuration for the [TemplateRuntime display card value](https://github.com/alexa/avs-device-sdk/blob/v1.10.0/Integration/AlexaClientSDKConfig.json#L144).
* A configuration file generator script, `genConfig.sh` is now included with the SDK in the **tools/Install** directory. `genConfig.sh` and it's associated arguments populate `AlexaClientSDKConfig.json` with the data required to authorize with LWA.
* Added Bluetooth A2DP source and AVRCP target support for Linux.
* Added Amazon for Business (A4B) support, which includes support for handling the new [RevokeAuthorization](https://developer.amazon.com/docs/alexa-voice-service/system.html#revokeauth) directive in the Settings interface. A new CMake option has been added to enable A4B within the SDK, `-DA4B`.
* Added locale support for IT and ES.
* The Alexa Communication Library (ACL), `CBLAUthDelegate`, and sample app have been enhanced to detect de-authorization using the new `z` command.
* Added `ExternalMediaPlayerObserver`, which receives notification of player state, track, and username changes.
* `HTTP2ConnectionInterface` was factored out of `HTTP2Transport` to enable unit testing of `HTTP2Transport` and re-use of `HTTP2Connection` logic.

**Bug Fixes**

* Fixed a bug in which `ExternalMediaPlayer` adapter playback wasn't being recognized by AVS.
* [Issue 973](#973) - Fixed issues related to `AudioPlayer` where progress reports were being sent out of order or with incorrect offsets.
* An `EXPECTING`, state has been added to `DialogUXState` in order to handle `EXPECT_SPEECH` state for hold-to-talk devices.
* [Issue 948](#948) - Fixed a bug in which the sample app was stuck in a listening state.
* Fixed a bug where there was a delay between receiving a `DeleteAlert` directive, and deleting the alert.
* [Issue 839](#839) - Fixed an issue where speech was being truncated due to the `DialogUXStateAggregator` transitioning between a `THINKING` and `IDLE` state.
* Fixed a bug in which the `AudioPlayer` attempted to play when it wasn't in the `FOREGROUND` focus.
* `CapabilitiesDelegateTest` now works on Android.
* [Issue 950](#950) - Improved Android Media Player audio quality.
* [Issue 908](#908) - Fixed compile error on g++ 7.x in which includes were missing.
@scotthea-amazon
Copy link
Contributor

Hello zeusshuang,

Release 1.10 includes a change to DialogUXStateAggregator to address this problem. If you can, please let us know whether or not this fixes the problem for you.

Thank you,
-SWH

@zeusshuang
Copy link
Author

Hi @scotthea-amazon ,

Thanks!! I will merge this release and try it.

@kclchan kclchan mentioned this issue Nov 9, 2018
8 tasks
@KrisBird
Copy link

I just installed 1.10 on a fresh Raspberry Pi3 Stretch and the response is cut in the start.

@kclchan
Copy link
Contributor

kclchan commented Dec 11, 2018

Hi @KrisBird, is your audio being output via HDMI? If so, could you try using a headphone jack instead?

@kjkh
Copy link

kjkh commented Jan 2, 2019

Hi @zeusshuang,

Are you still experiencing this issue after updating?

@KrisBird, are you still seeing the issue after switching audio output to the headphone jack?

@zeusshuang
Copy link
Author

Hi @kjkh ,
so far I did not see this problem in SDK1.11.

@KrisBird
Copy link

KrisBird commented Jan 3, 2019

I never found a reason for it not working and I ended up reinstalling. After that no problems :-)

@visayamv
Copy link
Contributor

visayamv commented Jan 4, 2019

Looks like this issue is no longer existent. I am closing this issue, please reopen if you need more assistance regarding this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants