node-red / node-red-nodes

Extra nodes for Node-RED
Other
988 stars 593 forks source link

node-red-node-multilang-sentiment: allow tokenization #1025

Closed chuank closed 9 months ago

chuank commented 1 year ago

Which node are you reporting an issue on?

node-red-node-multilang-sentiment

What are the steps to reproduce?

The current build only accepts an optional words object to override sentiment scores. It does not support the provision of tokenizers using a tokens object that the underlying multilang-sentiment Node.js package allows.

(am also happy to submit a PR for this; please let me know)

What happens?

Sentiment analysis of languages that require tokenization fails.

What do you expect to happen?

tokens should be passable to multilang-sentiment

Please tell us about your environment:

dceejay commented 1 year ago

Hi yes a PR would be much appreciated. Thanks