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

server response #54

Closed
dantodor opened this issue Dec 12, 2016 · 3 comments
Closed

server response #54

dantodor opened this issue Dec 12, 2016 · 3 comments
Assignees
Labels
type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@dantodor
Copy link

First of all, thanks for the great work!
I'm by all means no Python expert, I just wanted to try how this server works, and have encountered several issues down the road, some I could solve them, some not ... So, from the beginning:

  • numpy must be installed, maybe I missed that in the docs, but it threw error prior to install it
  • while trying to train, after installing MITIE following the steps in the docs, it won't work, at least in my setup, unless I give some extra params on the command line; the full command line I had to use is
    python -m rasa_nlu.train -c config.json -b mitie -d data/demo-rasa.json -p model
    Despite the fact mitie is configured in config.json, if I'm missing that on the command line, trainer will throw error by picking 'keyword' and saying it's not implemented. The same for the rest of the parameters.
  • after training is finished, I'm running the server as advised in the docs. Here I have two issues: I can't see the file with commands sent to the server, it's not being created, but more important, no matter what text I send for analysis, the server constantly answers like this:
    {"error": "Invalid parse parameter specified"}
    Command line is the advised one:
    curl -X POST -d '{"text":"I am looking for Chinese food"}' http://localhost:5000/parse

Can you please tell me what I'm doing wrong ? Where should I look for hints ?

Thanks a lot!

@amn41
Copy link
Contributor

amn41 commented Dec 12, 2016

Thanks for creating this issue! Let me try to help. Did you install the master brach? we've had some bugfixes recently so maybe git pull to update will fix some of these already.

  1. you're totally right - at the moment we do require numpy. See numpy is always required #55

  2. will make it easier to debug configuration options. but I think updating your clone should fix this.

  3. this looks like a naming clash. try replacing your json with '{"q":"I am looking for Chinese food"}' - I will chase up and update the docs

@amn41 amn41 added the type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. label Dec 12, 2016
@amn41 amn41 self-assigned this Dec 12, 2016
@amn41
Copy link
Contributor

amn41 commented Dec 12, 2016

see also issue #57

@amn41
Copy link
Contributor

amn41 commented Dec 12, 2016

cool both PRs are merged. Please clone latest version and let me know how it goes! closing now

@amn41 amn41 closed this as completed Dec 13, 2016
DominikRos pushed a commit that referenced this issue Feb 11, 2019
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

2 participants