classifier-reborn icon indicating copy to clipboard operation
classifier-reborn copied to clipboard

Published 20 hours ago • verified publisher icon jekyll

Multiple clasifications

Open gugaiz opened this issue 8 years ago • 4 comments

Hi I tried to do a test with multiple classifications (i.e. classifier = ClassifierReborn::Bayes.new 'Social', 'Email', 'Direct', 'Organic','Other') but it does not seems to work. I mean, even after training it always keep returning "Social" for different sentences. Should it work ok? Am I doing something wrong? or is this not supported yet?

gugaiz avatar Aug 25 '17 16:08 gugaiz

Which version are you using? The readme is focused on master, which I haven't released just yet.

Ch4s3 avatar Aug 31 '17 16:08 Ch4s3

I made it work for me. I forked and changed the way "Infinity" is being used, just in case you want to have a look to the change.

gugaiz avatar Aug 31 '17 16:08 gugaiz

@gugaiz could you link to what you're doing or open a pr here?

Ch4s3 avatar Sep 05 '17 13:09 Ch4s3

Ahh, I just looked at the fork https://github.com/gugaiz/classifier-reborn/commit/5fdd6b89bf08d6657cdb822dd479d86252661068 and I see what you're doing @gugaiz. This may be a bug WRT thresholds. I'd be happy to look at a pr.

Ch4s3 avatar Mar 06 '18 05:03 Ch4s3