visual-layer / fastdup

fastdup is a powerful free tool designed to rapidly extract valuable insights from your image & video datasets. Assisting you to increase your dataset images & labels quality and reduce your data operations costs at an unparalleled scale.
Other
1.56k stars 76 forks source link

[Bug]: combination of rotated bounding box and dinov2s do not work #177

Closed dbickson closed 1 year ago

dbickson commented 1 year ago

What happened?

df = pd.read_csv('mafat.csv')
fd2 = fastdup.create(input_dir='.', work_dir='output')
fd2.run(annotations=df, overwrite=True, license='XXXX', bounding_box='rotated', augmentation_additive_margin=25,
       verbose=False, ccthreshold=0.9, model_path='dinov2s', d=384)
Warning: This file includes artifacts from the DINOv2 repository, which is licensed under the
        Creative Commons Attribution-NonCommercial 4.0 International License.
        You are free to use this code as long as you provide attribution to the original author
        and use it in accordance with the terms of the license.
        For more information, please see: https://github.com/facebookresearch/dinov2/blob/main/LICENSE
FastDup Software, (C) copyright 2022 Dr. Amir Alush and Dr. Danny Bickson.
2023-05-01 06:40:21 [ERROR] When changing the default feature length d, using the default model_path,  you must run with run_mode=2 or run_mode=4 since we assume the user provides the binary features with their non default feature length d. Default length is 576 and user requested length 384 with run_mode 1
fastdup C++ error received:  When changing the default feature length d, using the default model_path,  you must run with run_mode=2 or run_mode=4 since we assume the user provides the binary features with their non default feature length d. Default length is 576 and user requested length 384 with run_mode 1

What did you expect to see?

No response

What version of fastdup were you runnning on?

0.925

What version of Python were you running on?

Python 3.8

Operating System

MacOSX 12.6.1

Reproduction steps

No response

Relevant log output

No response

Attach a screenshot [Optional]

No response

Contact Details [Optional]

No response

dbickson commented 1 year ago

To be fixed in 0.926, the combination of rotated+dino was not tested before