-
Notifications
You must be signed in to change notification settings - Fork 287
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
Tests look strange #47
Comments
What is also unclear to me is that in https://github.com/afedosenko/talon/blob/master/tests/signature/learning/featurespace_test.py
the last line contains 'john' which mean the last '0' should be '1' |
Hi @justafucker. Sorry for confusion and thanks for your interest / questions. Will try to explain them. The 1st test checks that There is a test that specifically checks that given But we definitely encourage you to submit a PR if you find tests / code confusing and wish to contribute / improve them. Regarding your 2nd question. The algo looks for lines like "John Doe" or "John" or "Doe" i.e. a line should end with extracted name or extracted name should be a detached word. This requirement might seem strange in respect to "john@example.com" but in general it helps to avoid false positives when extracted name happens to be some general sequence of chars that might occurs in a line. |
I'm looking at https://github.com/mailgun/talon/blob/master/tests/signature/learning/helpers_test.py
And don't understand why in
the expected result doesn't include 'serobnic'
The text was updated successfully, but these errors were encountered: