Skip to content

native serialization/deserialization for ivf#294

Open
razdoburdin wants to merge 1 commit intointel:dev/razdoburdin_streamingfrom
razdoburdin:streaming/ivf
Open

native serialization/deserialization for ivf#294
razdoburdin wants to merge 1 commit intointel:dev/razdoburdin_streamingfrom
razdoburdin:streaming/ivf

Conversation

@razdoburdin
Copy link
Contributor

@razdoburdin razdoburdin commented Mar 20, 2026

This PR introduce native serialization for IVF index.

Main changes are:

  1. New overload of svs::index::ivf::load_ivf_index accpting istream is introduced.
  2. Added related tests.
  3. save(std::ostream&) method for DynamicIVF is just a placeholder for now. Real implementation is expected later.

@razdoburdin razdoburdin marked this pull request as draft March 20, 2026 10:44
@razdoburdin razdoburdin marked this pull request as ready for review March 20, 2026 11:13
@razdoburdin razdoburdin requested a review from rfsaliev March 20, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ibhati ibhati ibhati approved these changes

@mihaic mihaic Awaiting requested review from mihaic mihaic is a code owner

@ahuber21 ahuber21 Awaiting requested review from ahuber21 ahuber21 is a code owner

@yuejiaointel yuejiaointel Awaiting requested review from yuejiaointel yuejiaointel is a code owner

@rfsaliev rfsaliev Awaiting requested review from rfsaliev

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@razdoburdin @ibhati