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

add update example #364

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add update example #364

wants to merge 1 commit into from

Conversation

wxyucs
Copy link
Collaborator

@wxyucs wxyucs commented Jan 22, 2025

No description provided.

@wxyucs wxyucs added kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) version/0.13 labels Jan 22, 2025
@wxyucs wxyucs self-assigned this Jan 22, 2025
@wxyucs wxyucs requested a review from jiaweizone as a code owner January 22, 2025 09:25
@wxyucs wxyucs requested review from LHT129 and inabao January 22, 2025 09:26
Signed-off-by: Xiangyu Wang <[email protected]>
@wxyucs wxyucs force-pushed the add-update-example branch from 0dcc7a1 to d56c126 Compare January 22, 2025 09:29
Copy link

codecov bot commented Jan 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##             main     #364      +/-   ##
==========================================
+ Coverage   90.81%   90.92%   +0.11%     
==========================================
  Files         133      133              
  Lines        8429     8445      +16     
==========================================
+ Hits         7655     7679      +24     
+ Misses        774      766       -8     
Flag Coverage Δ
cpp 90.92% <ø> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 95.75% <ø> (+0.91%) ⬆️
datacell 91.69% <ø> (-0.08%) ⬇️
index 91.11% <ø> (+0.04%) ⬆️
simd 81.63% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eeb0295...d56c126. Read the comment docs.

Copy link
Collaborator

@LHT129 LHT129 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

->Owner(false);

// try to update determines by the distance between the new vector and the old vector
if (auto update_vector = index->UpdateVector(update_id, update_dataset, false);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update_vector -> update_status ?

abort();
} else if (*update_vector) { /* updated, new vector is near to the old vector */
std::cout << "updated, new vector is near to the old vector" << std::endl;
} else { /* not update, new vector is far away from the old vector */
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q: default is in place update ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) size/L version/0.13
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants