telekom / nlu-bridge

MIT License
4 stars 5 forks source link

keeps all keys from Rasa entities in Rasa vendor predictions now #39

Closed kengelbrecht closed 1 year ago

kengelbrecht commented 1 year ago

This MR changes the test methods in Rasa2 and Rasa3 vendors to return all attributes of predicted entities instead of just "type", "start", and "end". Also, it removes nlubridge/vendors/rasa.py which apparently was a leftover from version 0 and was replaced by nlubridge/vendors/rasa2. In addition, it includes some fixes of incompatible dependencies, in particular datasets~=1.0 is required and for Rasa2 websockets~=10.4. There were also some install errors with Fasttext that I could not resolve so I removed those tests from the pipeline. I could successfully install and test on a Macbook. Will hopefully be resolved by the fasttext developers.