Skip to content

[C API] LeanVec training data handling refactoring - #378

Draft
rfsaliev wants to merge 1 commit into
dev/eglaser-leanvec-ood-capifrom
rfsaliev/c-api-leanvec-ood
Draft

[C API] LeanVec training data handling refactoring#378
rfsaliev wants to merge 1 commit into
dev/eglaser-leanvec-ood-capifrom
rfsaliev/c-api-leanvec-ood

Conversation

@rfsaliev

Copy link
Copy Markdown
Member

This PR contains suggestions to #358.

  • Refactor leanvec_training_data.hpp by adding an 'unimplemented' stub - allows to simplify svs_c.cpp code
  • Changed svs_leanvec_training_data_build() signature to use index bulder to take dims and threadpool configuration from - it allows end-user to manage threading for training data computation
  • Refactor StorageLeanVec to avoid leanvec_training_data.hpp dependency

* Refactor `leanvec_training_data.hpp` by adding an 'unimplemented' stub - allows to simplify svs_c.cpp code
* Changed svs_leanvec_training_data_build() signature to use index bulder to take `dims` and threadpool configuration from - it allows end-user to manage threading for training data computation
* Refactor `StorageLeanVec` to avoid `leanvec_training_data.hpp` dependency
@rfsaliev
rfsaliev force-pushed the rfsaliev/c-api-leanvec-ood branch from 2622fad to 4b4e661 Compare August 31, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant