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

A few more backend unit tests (trivial stuff) #1726

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

bitplane
Copy link
Collaborator

Just doing this bit by bit.

I've moved stuff about so it doesn't conflict with #605

Adds a bit more to #84 - it's some more examples and a bit more coverage, but doesn't actually run coverage or interfere with #605

@bitplane bitplane changed the title A few more unit tests (trivial stuff) A few more backend unit tests (trivial stuff) Feb 18, 2023
@bitplane bitplane marked this pull request as draft February 20, 2023 14:37
@bitplane
Copy link
Collaborator Author

Converted to draft for now, since some functions have been removed. I'll update in the next day or two

@bitplane bitplane force-pushed the more-testing branch 3 times, most recently from ba35948 to 1820d94 Compare February 22, 2023 06:46
@bitplane bitplane marked this pull request as ready for review February 22, 2023 06:47
@bitplane
Copy link
Collaborator Author

I changed the tests to do a simple test of the new cosine similarity functions, they're not great but still better than nothing.

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

Successfully merging this pull request may close these issues.

None yet

4 participants