Skip to content

Replace AlignedBoxWithSlice with plain Vec/Matrix where alignment is not needed #954

Description

@wuw92

Tracking issue for removing AlignedBoxWithSlice usage where the alignment is not actually needed, identified during the file-centric flow audit (#899).

Discussion

Originating proposal and maintainer confirmation in #899:

Scope boundary vs. #907

#907 tracks replacing AlignedBoxWithSlice with diskann-quantization::alloc::Poly for cases that do require alignment (e.g. direct I/O sector-aligned buffers).

This issue tracks the complementary cleanup: callers that use AlignedBoxWithSlice but whose consumers (distance computation through diskann-vector SIMD, or scalar PQ table lookups) don't actually require alignment. These can be replaced with plain Vec / Matrix.

Related PRs

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions