-
Notifications
You must be signed in to change notification settings - Fork 0
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
testing #24
Comments
@jhveem could you send me some of the arabic texts you used to test the accuracy of your model? Ideally I would have one for each readability level |
Yeah, I'm about to hop on a plane but I'll try to send it this evening.
…On Fri, Jul 12, 2019, 11:46 AM mjbriggs ***@***.***> wrote:
@jhveem <https://github.com/jhveem> could you send me some of the arabic
texts you used to test the accuracy of your model? Ideally I would have one
for each readability level
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#24?email_source=notifications&email_token=AKL6K2M6KZ4OPQWTFIXTDBDP7C7PZA5CNFSM4HMGPYN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ2NQCA#issuecomment-510973960>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKL6K2JZUHQ6AS54ZVG2P7DP7C7PZANCNFSM4HMGPYNQ>
.
|
Okay thanks |
@jhveem could you push your test files into master branch? They would go in src/test/resources/testFiles. |
As of writing this I have written unit tests for the server.raft package and part of the server.parser package. There are also some integration tests that perform web search operations for each language, document upload operations for english and german, and a raft integration test. |
We need automated testing. Let's use this thread to discuss/plan/prioritize.
The text was updated successfully, but these errors were encountered: