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

Check if text features are present #5199

Closed
tabergma opened this issue Feb 7, 2020 · 0 comments
Closed

Check if text features are present #5199

tabergma opened this issue Feb 7, 2020 · 0 comments
Assignees
Labels
type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@tabergma
Copy link
Contributor

tabergma commented Feb 7, 2020

Some users are getting intent predictions with high confidence for words that aren’t in their training data (e.g askdjflksjdf) since they upgraded to 1.6.1. They’re using word level featurization, it should be assigning the intent None.

They noticed that this check here : https://github.com/RasaHQ/rasa/blob/1.6.x/rasa/nlu/classifiers/embedding_intent_classifier.py#L825
doesn’t check whether there’s text features present any more like it used to:

if X.any() and intent_ids.size > 0:

Rasa version: 1.7.0
Python version: 3.6

@tabergma tabergma added the type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. label Feb 7, 2020
@tabergma tabergma self-assigned this Feb 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Development

No branches or pull requests

1 participant