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

Build errors when building with KittAi enabled #9

Closed
shikha-bansal opened this issue May 21, 2017 · 3 comments
Closed

Build errors when building with KittAi enabled #9

shikha-bansal opened this issue May 21, 2017 · 3 comments
Milestone

Comments

@shikha-bansal
Copy link

I get the following error when compiling 0.3 release with KittAi enabled:

In member function ‘virtual void alexaClientSDK::integration::AudioInputProcessorTest::SetUp()’:
/home/shikha.bansal/AVS/0.3/alexa-client-sdk-master/Integration/test/AudioInputProcessorIntegrationTest.cpp:624:22: error: converting to ‘std::unordered_set<std::shared_ptralexaClientSDK::avsCommon::sdkInterfaces::KeyWordDetectorStateObserverInterface >’ from initializer list would use explicit constructor ‘std::unordered_set<_Value, _Hash, _Pred, _Alloc>::unordered_set(std::unordered_set<_Value, _Hash, _Pred, _Alloc>::size_type, const hasher&, const key_equal&, const allocator_type&) [with _Value = std::shared_ptralexaClientSDK::avsCommon::sdkInterfaces::KeyWordDetectorStateObserverInterface; _Hash =

The 4th input parameter for function "create" in KittAiKeyWordDetector.cpp is a pointer to KeyWordDetectorStateObserverInterface . However the caller function Setup in AudioInputProcessorIntegration.cpp (line 615) , passes 4th parameter as { }.

Any ideas ?

@sanjayrd
Copy link
Contributor

Hi @shikha-bansal ,

Do you have GCC 4.9.2 by chance? Here's a local fix I believe should work:

Replace the {} parameter in the create function with std::unordered_set<std::shared_ptr<KeyWordDetectorStateObserverInterface>>() on line 620.

@shikha-bansal
Copy link
Author

Thanks @sanjayrd , fix works for me. I am able to compile.

@JamieMeyers
Copy link
Contributor

Hi @shikha-bansal,

We will be pushing the fix to mitigate the GCC regression for that version into the code base in the next release.

Thanks,
Jamie

kjkh pushed a commit that referenced this issue Jun 9, 2017
Changes in this update
-Implemented Sensory wake word detector functionality
-Removed the need for a std::recursive_mutex in MessageRouter
-Added AIP unit test
-Added handleDirectiveImmediately functionality to SpeechSynthesizer
-Added memory profiles for:
AIP
SpeechSynthesizer
ContextManager
AVSUtils
AVSCommon
-Bug fix for MultipartParser.h compiler warning
-Suppression of sensitive log data even in debug builds. Use cmake parameter -DACSDK_EMIT_SENSITIVE_LOGS=ON to allow logging of sensitive information in DEBUG builds
-Fix crash in ACL when attempting to use more than 10 streams
-Updated MediaPlayer to use autoaudiosink instead of requiring pulseaudio
-Updated MediaPlayer build to suppport local builds of GStreamer
-Fixes for the following Github issues:
#5
#8
#9
#10
#17
#24
@JamieMeyers JamieMeyers added this to the 0.4.1 milestone Jun 13, 2017
@xuqifu xuqifu mentioned this issue Jul 12, 2018
6 tasks
Guillaume0477 pushed a commit to Guillaume0477/avs-device-sdk that referenced this issue Sep 16, 2019
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

4 participants