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

Replace dialogflow with nlp.js (as option or entirely) #1

Open
1 of 2 tasks
dragfyre opened this issue May 29, 2021 · 3 comments
Open
1 of 2 tasks

Replace dialogflow with nlp.js (as option or entirely) #1

dragfyre opened this issue May 29, 2021 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@dragfyre
Copy link
Contributor

dragfyre commented May 29, 2021

Along with API changes, this requires replacing access tokens with keys

Ideally https://github.com/mishushakov/dialogflow-web-v2 would also be added.

Conversely, we might look into using https://www.npmjs.com/package/node-nlp instead of Dialogflow .

  • Replace apiai with (updated) dialogflow
  • Option to replace dialogflow with nlp.js
@dragfyre dragfyre added the enhancement New feature or request label May 29, 2021
@dragfyre dragfyre added this to the Release 1.1 milestone May 29, 2021
@brettz9
Copy link
Contributor

brettz9 commented May 29, 2021

It looked like they (nlp.js) had a number of dependencies in package.json where they split up the functionality. See the dependencies tab of https://www.npmjs.com/package/node-nlp to get links to the repos. Would it be relevant to investigate those also if you have not?

Re: converting from Dialogflow to nlp.js, I saw some for converting the "model" of Dialogflow--though GitHub-searching didn't turn up "entities": https://github.com/jpruden92/dialogflow-nlp-to-nlpjs or a fork https://github.com/levelbots/dialogflow-to-nlpjs

The main page of nlp.js mentioned entities fairly heavily in these subpages. Not sure if relevant:

In looking at your entities files to see what was significant, I noticed "synonyms" jumping out, and it seems there is an issue filed for that (if you wanted to subscribe): axa-group/nlp.js#40

Not sure if any of that surface-level examination turns up anything for ya.

@brettz9
Copy link
Contributor

brettz9 commented Jun 4, 2021

Not sure if you want to keep this open for the possibility of nlp but the move to Dialogflow is now complete on the main branch per commit 1d2f44c . Plan to give instructions on credentials shortly.

@brettz9 brettz9 changed the title Replace deprecated apiai (with nlp or DialogFlow) Replace dialogflow with nlp.js Jun 5, 2021
@brettz9 brettz9 changed the title Replace dialogflow with nlp.js Replace dialogflow with nlp.js (as option or entirely) Jun 5, 2021
@dragfyre
Copy link
Contributor Author

dragfyre commented Jun 6, 2021

Although the dialogflow migration is done, I think we can keep this issue open to discuss the possibility of moving away from third-party services altogether—seems like @sjscotti's workaround for the quote function may relate to that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants