ShirAmir / dino-vit-features

Official implementation for the paper "Deep ViT Features as Dense Visual Descriptors".

Home Page:https://dino-vit-features.github.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Use of the previous KMeans instance

mateusz-politycki-wttech opened this issue · comments

I wonder if using the previous KMeans instance at this point is intentional. I mean the part_algorithm was trained on normalized_all_fg_sampled_descriptors as opposed to common_part_algorithm, which is trained on normalized_all_common_sampled_descriptors.

_, common_part_labels = part_algorithm.index.search(normalized_all_common_descriptors.astype(np.float32), 1)

Hi @mateusz-politycki-wttech, thanks for pointing out the issue! it is indeed a bug.
it has been fixed and updated in the repo.

Due note that we recommend using the three stage version only on small sets.