Skip to content

Commit

Permalink
Add tests for reading/mmaping quantized embeddings
Browse files Browse the repository at this point in the history
Fixes #56.
  • Loading branch information
danieldk committed Sep 9, 2019
1 parent ba3fefe commit 72da5a8
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 0 deletions.
10 changes: 10 additions & 0 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,16 @@ def similarity_fifu(tests_root):
yield finalfusion.Embeddings(os.path.join(tests_root, "similarity.fifu"))


@pytest.fixture
def similarity_pq(tests_root):
yield finalfusion.Embeddings(os.path.join(tests_root, "similarity-pq.fifu"))


@pytest.fixture
def similarity_pq_mmap(tests_root):
yield finalfusion.Embeddings(os.path.join(tests_root, "similarity-pq.fifu"), mmap=True)


@pytest.fixture
def subword_fifu(tests_root):
yield finalfusion.Embeddings(os.path.join(tests_root, "subword.fifu"))
Expand Down
Binary file added tests/similarity-pq.fifu
Binary file not shown.
14 changes: 14 additions & 0 deletions tests/test_embeddings.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,20 @@ def test_embeddings(embeddings_fifu, embeddings_text, embeddings_text_dims):
embedding.embedding, embeddings_text_dims[embedding.word]), "FiFu and textdims embedding mismatch"


def test_embeddings_pq(similarity_fifu, similarity_pq):
for embedding in similarity_fifu:
embedding_pq = similarity_pq.embedding("Berlin")
assert numpy.allclose(embedding.embedding, embedding_pq,
atol=0.3), "Embedding and quantized embedding mismatch"


def test_embeddings_pq_mmap(similarity_fifu, similarity_pq_mmap):
for embedding in similarity_fifu:
embedding_pq = similarity_pq_mmap.embedding("Berlin")
assert numpy.allclose(embedding.embedding, embedding_pq,
atol=0.3), "Embedding and quantized embedding mismatch"


def test_embeddings_with_norms_oov(embeddings_fifu):
assert embeddings_fifu.embedding_with_norm(
"Something out of vocabulary") is None
Expand Down

0 comments on commit 72da5a8

Please sign in to comment.