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

Add test cases for Bayesian Networks using reference implementation. #7

Open
markdouthwaite opened this issue May 14, 2020 · 2 comments
Assignees
Labels
good first issue Good for newcomers help wanted Extra attention is needed testing-enhancement This issue relates to improving the package's test suite.
Milestone

Comments

@markdouthwaite
Copy link
Owner

The existing test suite is essentially non-existent. Given the ongoing changes to the model interface, it's essential there's a set of tests that give decent assurance that the behaviour of the models remain correct.

@markdouthwaite markdouthwaite added help wanted Extra attention is needed good first issue Good for newcomers testing-enhancement This issue relates to improving the package's test suite. labels May 14, 2020
@markdouthwaite markdouthwaite added this to the 0.2.0 milestone May 14, 2020
@markdouthwaite markdouthwaite self-assigned this May 14, 2020
@github-actions
Copy link

Hey! Thanks for raising an issue and contributing to Apogee!

@markdouthwaite
Copy link
Owner Author

This should include a comparison to Scikit-Learn's Naive Bayes implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed testing-enhancement This issue relates to improving the package's test suite.
Projects
None yet
Development

No branches or pull requests

1 participant