-
Notifications
You must be signed in to change notification settings - Fork 109
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
Default hash cleanup #45
Conversation
This mod makes it soe that you can train categories that have not been previously defined in the constructor.
@@ -6,23 +6,26 @@ | |||
|
|||
module ClassifierReborn | |||
class Bayes | |||
class CategoryNotFound < StandardError; end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent! Please suffix this with Error
or Exception
.
What do you think about CategoryNotFoundError = Class.new(StandardError)
to create them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add Error
. Is there a compelling reason to choose one exception definition over another?
I'll take a look this weekend 👍 |
Few things got done here: