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

Strip rasa as a dependency #21

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Strip rasa as a dependency #21

wants to merge 4 commits into from

Conversation

haz
Copy link
Member

@haz haz commented Aug 25, 2023

Closes #15 .

  • When building and running the docker (see the README on the front page), you will get an exception thrown about a missing NLUOutcomeDeterminer.
  • Currently, spacy is still part of the NLUOutcomeDeterminer, but it is only used to extract entities. Something is still needed to extract intents.
  • When the new outcome determiner is added, the docker will need be updated as well as the conversation simulation modules.

@haz haz marked this pull request as draft August 25, 2023 15:12
@beckydvn beckydvn added bug Something isn't working help wanted Extra attention is needed and removed bug Something isn't working labels Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Streamline the Docker builds
2 participants