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

Update neon-mq-connector dependency to stable spec (#43) #44

Closed
wants to merge 40 commits into from

Conversation

github-actions[bot]
Copy link

Co-authored-by: Daniel McKnight [email protected]

NeonDaniel and others added 30 commits February 16, 2023 17:05
* Update automation to shared repository
Add license test automation

* Update neon-utils dependency extras
Patching automation error
* Add CLI endpoints to interact with API and LLM endpoints

* Add longer timeout for LLM CLI
Update messagebus dependency to ovos-bus-client

* Fix config handling bug

* Add option for weather CLI to select an endpoint

* Add options to LLM help text

* Print MQ server config when using CLI entrypoints

* Annotate missing CLI methods

* Implement coupons util
Implement script util
Update docstrings

* Loosen pyyaml dependency to work around yaml/pyyaml#724
Update mq connector dependency to include routing fixes
* Add NeonVoiceClient class for minimal remote audio client

* Add CLI entrypoint for listener
Add WW confirmation sound
Update logging and bugfixes around MQ event handling
* Initial Gradio web UI implementation with Dockerfile

* Initial outline of settings with UI

* Annotate web_client
Move hard-coded params to configuration
Update README to include configuration example
Update Docker default configuration
Add Docker automation

* wip tts/stt and styling

* add stt/tts, rearrange interface

* Address feedback from #25
Update documentation
Add missing system deps to Dockerfile

* Patch audioread missing license (MIT) https://github.com/beetbox/audioread/blob/main/LICENSE

* Refactor `docker` dependencies to `gradio`
Cleanup logging
Refactor to resolve warnings
Resolve missing directory exception in audio input handling

* Fix STT language handling

---------

Co-authored-by: Daniel McKnight <[email protected]>
Co-authored-by: mikejgray <[email protected]>
* Prevent sending an input until the previous response has been handled
This would ideally use a queue but that will require using a different UI since the gradio ChatBot expects each input to return a value synchronously
Relates to #26

* Implement gradio State to track a session ID
Update handling so TTS responses are attached to a specific browser session

* Implement session-specific profile settings

* Add remaining user profile params to UI

---------

Co-authored-by: Daniel McKnight <[email protected]>
Handle MQ connection errors with useful log in CLI
Update default config for Docker

Co-authored-by: Daniel McKnight <[email protected]>
* Remove audio resampling moved to neon-speech NeonGeckoCom/neon_speech#180
Closes #28

* Add timing context and logging to go with:
NeonGeckoCom/neon_speech#181
NeonGeckoCom/neon_audio#154

* More timing metrics and logging

* Refactor timing and add debug log
Prevent response error from affecting next input

---------

Co-authored-by: Daniel McKnight <[email protected]>
Relocate audio input box next to `Submit` button
Clear audio input upon response to input
Move `Play TTS` button to visually match line above
Updates default web UI labels in Docker config
Closes #30
Closes #29

Co-authored-by: Daniel McKnight <[email protected]>
NeonDaniel and others added 7 commits November 22, 2023 17:05
Fix typo in audio input gradio handling

Co-authored-by: Daniel McKnight <[email protected]>
* Add handler for Neon core profile updates
Add `_languages` dict with handling of language API responses
Updates default config to include languages supported in default Neon Core installation
Closes #35

* Disable yet-to-be-implemented language API by default
Catch MQ connection error exceptions in CLI entrypoint
Override default location in Docker system config

* Add configuration note RE language support

* Add cli entrypoint to get languages
Update Docker config to default to use language API

* Update language support to use combined API
Ensure default lang populates settings fields

---------

Co-authored-by: Daniel McKnight <[email protected]>
@NeonDaniel NeonDaniel closed this Dec 11, 2023
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.

1 participant