Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iOS Voice Assistant #2

Merged
merged 26 commits into from
Jun 5, 2024
Merged

iOS Voice Assistant #2

merged 26 commits into from
Jun 5, 2024

Conversation

ErisMik
Copy link
Contributor

@ErisMik ErisMik commented May 31, 2024

No description provided.

@ErisMik ErisMik force-pushed the ios branch 2 times, most recently from c68f8c1 to d256117 Compare May 31, 2024 23:01
@ErisMik ErisMik requested a review from laves June 3, 2024 16:50
@ErisMik ErisMik assigned laves and ksyeo1010 and unassigned laves Jun 3, 2024
@ErisMik ErisMik requested a review from laves June 4, 2024 20:55
@ksyeo1010
Copy link
Contributor

I am getting some failed to register bundle identifier errors, does this happened to you?

@ErisMik
Copy link
Contributor Author

ErisMik commented Jun 4, 2024

I am getting some failed to register bundle identifier errors, does this happened to you?

Yeah this happens with all our demos, you have to change the bundle identifier to something like ai.picovoice.LLMVoiceAssistantDemo.Kwangsoo or something on your machine.

@ErisMik
Copy link
Contributor Author

ErisMik commented Jun 4, 2024

@laves Can't seem to reply to your comment. Fixed it using the callbacks correctly, we now only record when needed. Also fixed a bug with not flushing the warmupPCM if it never reaches the 1 second mark.

@ErisMik ErisMik requested a review from laves June 4, 2024 23:59
@laves laves merged commit 1c1765d into main Jun 5, 2024
3 checks passed
@laves laves deleted the ios branch June 5, 2024 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants