5 Commits
Author SHA1 Message Date
PiechockiF 7744239dc5 Add getter/setter for balanced_bins in PCAMatrix C API issue #4617 (#4630)
Summary:
Adds getter and setter for `balanced_bins` field in PCAMatrix through the C API.

## Motivation
Enables access to the `balanced_bins` parameter from language bindings (e.g., Rust).

Fixes https://github.com/facebookresearch/faiss/issues/4617

## Changes
- Added `FAISS_DECLARE_GETTER` and `FAISS_DECLARE_SETTER` for `balanced_bins` in `c_api/VectorTransform_c.h`
- Added `DEFINE_GETTER` and `DEFINE_SETTER` implementations in `c_api/VectorTransform_c.cpp`

## Testing
- Code follows existing patterns in the C API
- No API changes to existing functionality

Pull Request resolved: https://github.com/facebookresearch/faiss/pull/4630

Reviewed By: mnorris11

Differential Revision: D85156198

Pulled By: alibeklfc

fbshipit-source-id: 4c2be05f0d45fdc9cf1541d980646da3253e7eea
2025-10-21 14:00:45 -07:00
Michael Norris eff0898a13 Enable linting: lint config changes plus arc lint command (#3966)
Summary:
Pull Request resolved: https://github.com/facebookresearch/faiss/pull/3966

This actually enables the linting.

Manual changes:
- tools/arcanist/lint/fbsource-licenselint-config.toml
- tools/arcanist/lint/fbsource-lint-engine.toml

Automated changes:
`arc lint --apply-patches --take LICENSELINT --paths-cmd 'hg files faiss'`

Reviewed By: asadoughi

Differential Revision: D64484165

fbshipit-source-id: 4f2f6e953c94ef6ebfea8a5ae035ccfbea65ed04
2024-10-22 09:46:48 -07:00
Michael Norris 56a383f253 Resolve "duplicate-license-header": Find and replace duplicate license headers (#3967)
Summary:
Pull Request resolved: https://github.com/facebookresearch/faiss/pull/3967

This diff just finds and replaces duplicate license headers.

See the errors for "duplicate-license-header" in D64429711 under "linter-coverage-verification" signal.

Reviewed By: asadoughi

Differential Revision: D64484123

fbshipit-source-id: 906e8baa3a11a3bbee174a03dcc27681f9fd78c2
2024-10-21 08:26:22 -07:00
Alexander Andreev a87930111e Classes inherited from VectorTransform for c_api (#1869)
Summary:
inherite from  LinearTransform doesn't work here.

Pull Request resolved: https://github.com/facebookresearch/faiss/pull/1869

Reviewed By: beauby

Differential Revision: D28345866

Pulled By: mdouze

fbshipit-source-id: 277dd421213a91c07ed41d7b23002840cc5cfa1f
2021-05-12 07:34:39 -07:00
Alexander Andreev d640c6fcda Impl IndexPreTransform for c_api (#1816)
Summary:
This PR extends c_api for IndexPreTransform

Pull Request resolved: https://github.com/facebookresearch/faiss/pull/1816

Reviewed By: beauby

Differential Revision: D27904597

Pulled By: mdouze

fbshipit-source-id: b54dfffcc97879fdf66f9a8a26e9b7840a2e97f2
2021-04-22 05:27:36 -07:00