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

Support for MQ-based Chatbots #158

Draft
wants to merge 123 commits into
base: master
Choose a base branch
from
Draft

Support for MQ-based Chatbots #158

wants to merge 123 commits into from

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Nov 24, 2021

Added dynamic support for MQ-based chatbot connections

NeonKirill and others added 2 commits November 24, 2021 10:14
* 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 NeonDaniel closed this Nov 24, 2021
@NeonDaniel NeonDaniel reopened this Nov 24, 2021
@NeonDaniel NeonDaniel changed the title Support for MQ-based Chatbots (#149) Support for MQ-based Chatbots Nov 24, 2021
@NeonDaniel NeonDaniel mentioned this pull request Dec 13, 2021
NeonDaniel and others added 30 commits November 29, 2023 10:43
* 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]>
* 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]>
Reduce info log to debug

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants