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

[FEA] Helpers and CodePacker for IVF-PQ #1826

Merged
merged 89 commits into from
Nov 17, 2023

Conversation

tarang-jain
Copy link
Contributor

@tarang-jain tarang-jain commented Sep 13, 2023

  • Codepacking for compressed on-device (flat) PQ codes
  • Testing

tarang-jain and others added 30 commits June 30, 2023 17:03
* externally modifying the index without going through the build stage.
*
* @tparam IdxT
* @param[in] res
Copy link
Member

Choose a reason for hiding this comment

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

As mentione- please provide usage examples for ALL public facing functions.

Copy link
Member

@cjnolet cjnolet left a comment

Choose a reason for hiding this comment

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

LGTM. Let's do a follow-up for raft::neighbors::ivf_flat::helpers::reset_index(), though.

@@ -106,5 +109,15 @@ void unpack(
res, list_data, veclen, offset, codes);
}
} // namespace codepacker

template <typename T, typename IdxT>
Copy link
Member

Choose a reason for hiding this comment

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

Why doesn't this have a doc / usage examples?

@cjnolet
Copy link
Member

cjnolet commented Nov 16, 2023

/ok to test

@cjnolet cjnolet closed this Nov 16, 2023
@cjnolet cjnolet reopened this Nov 16, 2023
@cjnolet
Copy link
Member

cjnolet commented Nov 16, 2023

/ok to test

@tarang-jain
Copy link
Contributor Author

/ok to test

1 similar comment
@cjnolet
Copy link
Member

cjnolet commented Nov 16, 2023

/ok to test

@cjnolet
Copy link
Member

cjnolet commented Nov 17, 2023

/ok to test

Copy link
Member

@cjnolet cjnolet left a comment

Choose a reason for hiding this comment

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

LGTM!

@cjnolet
Copy link
Member

cjnolet commented Nov 17, 2023

/merge

@cjnolet
Copy link
Member

cjnolet commented Nov 17, 2023

/ok to test

@rapids-bot rapids-bot bot merged commit 52a5e4c into rapidsai:branch-23.12 Nov 17, 2023
61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp feature request New feature or request non-breaking Non-breaking change
Projects
Development

Successfully merging this pull request may close these issues.

2 participants