-
Notifications
You must be signed in to change notification settings - Fork 0
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
Support for MQ-based Chatbots #158
Draft
github-actions
wants to merge
123
commits into
master
Choose a base branch
from
dev
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Changed structure and added v2 for ChatBot class * dropped not needed method * Moved common chat bot properties to abstract chatbot class * Moved chatbot methods to utils * Moved chatbot methods to utils * added docstring * typo fix * Dropped chatbot dependency for Parlai Bot * Updated usage of NeonBot and ParlAI Bot classes * Added mq-based handle shout functionality for chatbot v2 * Added docstring to new methods and responded shout id handling * Updated requirements.txt * Added current conversation on chatbot declaration * Added klat connector to the list of dependencies * Changed required version * Small fix * Added utilities for mq-based running of chatbots * Fixed imports * import fix * small fix * rollback to stable imports structure * Some more logging * Test * small fix * bugfix * Added support for dynamic init Co-authored-by: kirill <[email protected]>
NeonDaniel
changed the title
Support for MQ-based Chatbots (#149)
Support for MQ-based Chatbots
Nov 24, 2021
Closed
Generic Version Utility
ParlAI troubleshooting
* Update package name to `neon-chatbot-core` to resolve PyPI name conflict Fix relative path refs in setup.py * Update package name in license tests --------- Co-authored-by: Daniel McKnight <[email protected]>
* Add entrypoint to start chatbot by entrypoint * Fix typo in added entrypoint * Resolve circular import * Fix typo in added entrypoint * Fix typo in added entrypoint * init log to reduce debug output * Troubleshooting log init * Fix typo in log init config --------- Co-authored-by: Daniel McKnight <[email protected]>
Co-authored-by: Daniel McKnight <[email protected]>
* Fix invalid reference in ParlaiBot class init * Remove `self.log` from `ParlaiBot` to be compatible with `ChatBotABC` property --------- Co-authored-by: Daniel McKnight <[email protected]>
* Refactor CLI entrypoints into a submodule Refactor init to avoid unused imports with backwards-compat wrappers Refactor tests into integration and unit tests Add unit tests * Troubleshooting test failures * Refactor tests for proper collection * Troubleshoot GH test failures * Update Messagebus import in tests * Add backwards-compat. imports for `chatbot_core.utils` * Finish CLI refactor Update debug util to use Mach server locally * Cleanup changes * Add docs and function annotations Remove added CLI entrypoints for deprecated functions * Increase test coverage Mark util methods for deprecation * Add `lang` extras to unit test setup * Troubleshooting automated test failure * Fix typo in setup.py extra deps * Resolve warnings Add method type annotation * Mark `logger` as deprecated and remove references --------- Co-authored-by: Daniel McKnight <[email protected]>
* Add support for chatbot-specific configuration Update module init to set default config envvars Add config documentation to README.md * Refactor init Update `ChatBot.log` to use bot_id for log name instead of class name * Minor logging refactor * Cleanup changes Update documentation --------- Co-authored-by: Daniel McKnight <[email protected]>
* Implement v2 integration tests from `chatbots` repository Fix v1 ChatBot to define `bot_type` Add CLI entyrpoint for local discussion session Implement utils from `chatbots` repository * Update tests Fix prompter behavior in local tests * Troubleshooting skipped test * Troubleshooting skipped test * Revert removed test case * Outline tests for added methods Continue local test CLI * Update config handling * Fix prompter response handling --------- Co-authored-by: Daniel McKnight <[email protected]>
Co-authored-by: Daniel McKnight <[email protected]>
* Add test coverage for Chatbot init * Remove Python 3.10-3.11 tests needing klat-connector support --------- Co-authored-by: Daniel McKnight <[email protected]>
* Copy `LOG` for each running chatbot * Fix log init to get per-bot loggers * Downgrade proctor ping-pong log to debug * Update chatbot log to retain usual formatting * Fix log name formatting * Update default LOG.name * Update log unit tests --------- Co-authored-by: Daniel McKnight <[email protected]>
* Added async consumers support * Dropped support for Python 3.7 * Updated requirements * bumped gh actions to the latest supported versions * removed support for python 3.7 * added pre-fetching punkt_tab dependency in nltk * Small fix to conflicting artifact upload
- Added handling of "announce_invitation" instead of "skip_announcement" - Introduced "FACILITATOR" bot type
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added dynamic support for MQ-based chatbot connections