-
Notifications
You must be signed in to change notification settings - Fork 94
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
Make B.E.N.J.I. recognize positions of "stream" and "play" #83
Comments
Nice spotting the bug, @rohanricky ! |
As this can be a very broad issue, can we start off using NLP gradually, maybe using nltk? We can do basic tokenizing. |
Good thinking @kb-studios ! But tokenizing requires a knowledge of NLP. But though won't fix for sometime, this is a good issue for discussion |
Yeah we have to use nlp at the end of the day.😀 So as of now it's limited |
For now we can reverse it's order to get it working or since we are using download and play at starting only we can use .startwith() for this also. This is just a quick fix eventually we can work on nlp. |
You are right @rohanricky in this sense. We can use that temporarily. You can work on this issue. but we are not closing it |
Since youtube play feature is above download it is causing few bugs errors.
For example, if I type download Coldplay Paradise, it contains play so it is playing the video. Similarily, Chainsmokers Bloodstream contains stream.
The text was updated successfully, but these errors were encountered: