Intent classification

[pep8speaks]

Hello @poojithamiryala! Thanks for updating this PR.

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2020-04-25 14:48:26 UTC

[satyammittal]

A) Can you only keep changes only related to intent classification. like you can remove .idea folder.

B) Please fix pep8 issues.

[poojithamiryala]

Removed .idea folder and resolved pip8 issues

[satyammittal]

looks good.

can you integrate this with current chatbot.py.

[pravalikavis]

So to integrate to chatbot.py do we need to train all the categories or use our model to decide whether it is music/weather/restaurant/None, and if it gives none we continue the normal flow of else if.

[satyammittal]

@pravalikavis Yes, this approach looks fine for initial start.

[poojithamiryala]

Integrated intent classification file with the chatbot

[satyammittal]

@poojithamiryala Please fix the pep8 issues.

[poojithamiryala]

Fixed pep8 issues

[satyammittal]

@poojithamiryala I could still see some code space issues listed by our code style saviour @pep8speaks . Please pull latest changes again as I recently merged a new change. Thanks.

[poojithamiryala]

I tried fixing most of the pep8 issues . Can u please have a look and tell me if there are any?

[satyammittal]

@poojithamiryala can you pull latest changes and fix conflicts.

[satyammittal]

@poojithamiryala can you replace json files with json.zip. and read zip files instead. https://stackoverflow.com/questions/40824807/reading-zipped-json-files

[poojithamiryala]

I have compressed JSON files and read data using zip files .

[satyammittal]

Nice start.