Skip to content

Validate SparseVector dictionary indices - #167

Closed
AryanRG wants to merge 1 commit into
pgvector:masterfrom
AryanRG:fix/sparsevector-index-bounds
Closed

AryanRG wants to merge 1 commit into
pgvector:masterfrom
AryanRG:fix/sparsevector-index-bounds

Conversation

@AryanRG

@AryanRG AryanRG commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • reject dictionary-backed SparseVector indices outside the supplied dimensions
  • add regression coverage for negative and upper-bound indices

Verification

  • 68 core tests passed with NumPy and SciPy enabled
  • mypy passed for the full repository
  • ty passed for the full repository
  • pycodestyle passed for the changed files

Database-backed integration tests were not run locally because PostgreSQL was not available.

@ankane ankane closed this in 4003693 Oct 16, 2026
@ankane

ankane commented Oct 16, 2026

Copy link
Copy Markdown
Member

Hi @AryanRG, thanks for the PR. Will include this and other validations in 0.6.0.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants