Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple clasifications #165

Open
gugaiz opened this issue Aug 25, 2017 · 4 comments
Open

Multiple clasifications #165

gugaiz opened this issue Aug 25, 2017 · 4 comments

Comments

@gugaiz
Copy link

gugaiz commented Aug 25, 2017

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?

@Ch4s3
Copy link
Member

Ch4s3 commented Aug 31, 2017

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

@gugaiz
Copy link
Author

gugaiz commented Aug 31, 2017

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.

@Ch4s3
Copy link
Member

Ch4s3 commented Sep 5, 2017

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

@Ch4s3
Copy link
Member

Ch4s3 commented Mar 6, 2018

Ahh, I just looked at the fork gugaiz@5fdd6b8 and I see what you're doing @gugaiz. This may be a bug WRT thresholds. I'd be happy to look at a pr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants