search

bahaidev / bahaibot

A Bahá'í bot for Discord. QnA chatbot functionality, scripture reader, and integration with Bahaipedia.
2 stars 0 forks source link

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

Open dragfyre opened 3 years ago

dragfyre commented 3 years ago

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 .

brettz9 commented 3 years ago

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): https://github.com/axa-group/nlp.js/issues/40

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

brettz9 commented 3 years ago

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 1d2f44c1d61de0835847273739f95ef493242634 . Plan to give instructions on credentials shortly.

dragfyre commented 3 years ago

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.