usearch icon indicating copy to clipboard operation
usearch copied to clipboard

Bug: getting irrelevant closest items when using dtype='bf16' (default) instead of dtype='f16'

Open alexyalunin opened this issue 1 year ago • 7 comments

Describe the bug

Just updated library from 2.12.0 to 2.15.3 Now I'm getting irrelevant closest items.

I expect for an anchor item to be closest to it by any metric (I use cos), you can see in Expected behavior screenshot. Now after update anchor item is not closest + I get a lot of irrelevant items for it (see the screenshot).

It turns out the difference is in dtype='bf16' which is default in new version. After reverting it back to dtype='f16' I get an expected behabior.

Screenshot 2024-10-14 at 17 03 45

Steps to reproduce

Here is the code I use

from usearch.index import Index
item_count, dimension = train_matrix.shape
index = Index(
    ndim=dimension, # Define the number of dimensions in input vectors
    metric='cos', # Choose 'l2sq', 'haversine' or other metric, default = 'ip'
    dtype='bf16', # Quantize to 'f16' or 'i8' if needed, default = 'f32'
    # connectivity=16, # Optional: Limit number of neighbors per graph node
    # expansion_add=128, # Optional: Control the recall of indexing
    # expansion_search=64, # Optional: Control the quality of the search
    # multi=False, # Optional: Allow multiple vectors per key, default = False
)

_ = index.add(list(range(item_count)), train_matrix)

k = 100
res = index.search(train_matrix, count=k+1)  # +1 because same product is closest
train_similars = train_ids[res.keys]

Expected behavior

using dtype='f16' Screenshot 2024-10-14 at 17 05 53

USearch version

2.15.3

Operating System

Ubuntu 20.04.6 LTS

Hardware architecture

x86

Which interface are you using?

Python bindings

Contact Details

[email protected]

Are you open to being tagged as a contributor?

  • [ ] I am open to being mentioned in the project .git history as a contributor

Is there an existing issue for this?

  • [X] I have searched the existing issues

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

alexyalunin avatar Oct 14 '24 14:10 alexyalunin

Hi @alexyalunin! Did you only change the dtype in the constructor or somewhere else?

ashvardanian avatar Oct 14 '24 14:10 ashvardanian

Can you please log the type of the input matrix and the hardware_capabilities of the index?

ashvardanian avatar Oct 14 '24 14:10 ashvardanian

train_matrix.dtype dtype('float32')

Idk how to log hardware_capabilities, here is index

usearch.Index

  • config -- data type: ScalarKind.BF16 -- dimensions: 64 -- metric: MetricKind.Cos -- multi: False -- connectivity: 16 -- expansion on addition :128 candidates -- expansion on search: 64 candidates
  • binary -- uses OpenMP: 0 -- uses SimSIMD: 1 -- supports half-precision: 1 -- uses hardware acceleration: haswell
  • state -- size: 945,655 vectors -- memory usage: 406,848,192 bytes -- max level: 4 --- 0. 945,655 nodes --- 1. 58,816 nodes --- 2. 3,632 nodes --- 3. 264 nodes --- 4. 24 nodes

alexyalunin avatar Oct 14 '24 14:10 alexyalunin

I didn't change dtype of index by myself. In the example I changed dtype to bf16 to reproduce the problem

alexyalunin avatar Oct 14 '24 15:10 alexyalunin

I have just rebuilt the index 5 times in a loop and look at 5 anchor items (i.e. 25 examples).

It seems like even with f16 the problem still exists, it is just less often than with bf16. It also seems like with the old version (2.12.0) the problem doesn't exist, so I guess I will just stay with an old version.

alexyalunin avatar Oct 14 '24 15:10 alexyalunin

Interesting 🤔 Thank you, @alexyalunin! I will look into it!

ashvardanian avatar Oct 14 '24 17:10 ashvardanian

Found the issue in the underlying SimSIMD library. Investigating. Hope to merge within 24h.

ashvardanian avatar Oct 14 '24 20:10 ashvardanian

Hi @alexyalunin! I suppose the issue should be long resolved by now. Feel free to open otherwise 🤗

ashvardanian avatar Jun 08 '25 05:06 ashvardanian