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

Rasa X's NLU Post API Should Support Markdown #4076

Open
rgstephens opened this issue Jul 22, 2019 · 13 comments

Comments

@rgstephens
Copy link
Contributor

commented Jul 22, 2019

Description of Problem:
The stated direction of Rasa is to use Markdown not JSON but the http POST API for training data supports only JSON. This API call also lacks symmetry with the adding of stories via it's POST call which takes a markdown file.

Overview of the Solution:
There should be a post that allows upload of markdown like the stories post.

@msamogh

This comment has been minimized.

Copy link
Member

commented Jul 23, 2019

Thanks for raising this issue, @MetcalfeTom will get back to you about it soon.

@MetcalfeTom

This comment has been minimized.

Copy link
Contributor

commented Jul 23, 2019

Hi @rgstephens,

Is this for Rasa X? The Rasa NLU HTTP API should already support both file types.

@rgstephens

This comment has been minimized.

Copy link
Contributor Author

commented Jul 23, 2019

Yes, for Rasa X.

@MetcalfeTom MetcalfeTom added the Rasa X label Jul 23, 2019

@MetcalfeTom

This comment has been minimized.

Copy link
Contributor

commented Jul 23, 2019

Seems valuable, I'll add it to our maintenance

@rgstephens

This comment has been minimized.

Copy link
Contributor Author

commented Aug 5, 2019

I just realized that the GET call also only returns json. It would be nice to support markdown there also. Probably applies PUT too.

@msamogh

This comment has been minimized.

Copy link
Member

commented Aug 6, 2019

I believe you can upload NLU data in bulk using Markdown to Rasa X as of now. Only single examples should be restricted to JSON.

@msamogh

This comment has been minimized.

Copy link
Member

commented Aug 6, 2019

I agree about the GET call, though.

@rgstephens

This comment has been minimized.

Copy link
Contributor Author

commented Aug 6, 2019

@msamogh I assume you're referring to the Rasa X UI import button which does allow you to import markdown. I'm referring to the API POST call which supports a json payload only.

@rgstephens

This comment has been minimized.

Copy link
Contributor Author

commented Aug 6, 2019

@MetcalfeTom can you give an estimate on where this feature is? I'm working on one of my bots that has NLU data in six markdown files with 300 intents.

I'd like to import this with the API and the Rasa X NLU Import has issues described in #3580. The workaround I'm using is to merge my markdown files and use the UI but both this API issue and the UI import issue need to be addressed.

@msamogh

This comment has been minimized.

Copy link
Member

commented Aug 7, 2019

@rgstephens

This comment has been minimized.

Copy link
Contributor Author

commented Aug 8, 2019

@msamogh The payload for the PUT call you referenced is also json. This is a request to support the markdown format that has been recommended by Rasa for the past 9 months or so.

@msamogh msamogh changed the title NLU Post API Should Support Markdown Rasa X's NLU Post API Should Support Markdown Aug 8, 2019

@msamogh

This comment has been minimized.

Copy link
Member

commented Aug 8, 2019

@rgstephens You just have to set the Content-Type to text/markdown. It's also documented in the Rasa X HTTP API.

@rgstephens

This comment has been minimized.

Copy link
Contributor Author

commented Aug 9, 2019

Thanks, I didn't notice the text/markdown option. That's a good workaround for me until POST with markdown is available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.