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

Sentiment NLU middleware #45

Closed
timkinnane opened this issue Jun 24, 2018 · 2 comments
Closed

Sentiment NLU middleware #45

timkinnane opened this issue Jun 24, 2018 · 2 comments
Labels
DOCS Doc missing / incorrect FEAT Feature request / proposal
Milestone

Comments

@timkinnane
Copy link
Contributor

Not sure yet if sentiment should be handled as part of NLU adapter by default, but some services might not provide it and some sentiment services won't provide intent/entities either.

First draft demonstrating sentiment should probably be on NLU middleware, implementing one or multiple of the following:

https://medium.com/@sifium/top-five-emotional-sentiment-analysis-apis-116cd8d42055

This is probably more for a demo than a core feature.

@timkinnane timkinnane added the FEAT Feature request / proposal label Jun 24, 2018
@timkinnane timkinnane added this to the Minor milestone Jun 24, 2018
@timkinnane
Copy link
Contributor Author

@timkinnane timkinnane modified the milestones: Minor, Alpha Jun 24, 2018
@timkinnane timkinnane added the DOCS Doc missing / incorrect label Jun 24, 2018
@timkinnane
Copy link
Contributor Author

This is resolved now by recent updates which settled on a universal NLU result that can contain sentiment or any other data. Is implemented by the Watson Tone adapter and will be illustrated in #52.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DOCS Doc missing / incorrect FEAT Feature request / proposal
Projects
None yet
Development

No branches or pull requests

1 participant