DanielRapp / twss.js

A node.js "that's what she said" classifier
MIT License
694 stars 34 forks source link

Changing numWordsInNgram breaks classification if already trained #14

Closed thomwiggers closed 10 years ago

thomwiggers commented 10 years ago

Due to the hash not including all of the parameters which are used in calculating the probabilities, changing the n of n-grams will not cause a re-evaluation of the probabilities with this new setting. Because the incoming instance (prompt) is being split by the new numWordsInNgrams it breaks classification.

DanielRapp commented 10 years ago

Fantastic, thanks!