Normalization should not be done in the wrapper

[DavidGOrtega]

Normalization should be done on the C++ side (if any, my vectors come out from my models already normalised).
If not every time the C++ code is used by another programming languages (i.e. nodejs) normalization has to be reimplemented

hnswlib/python_bindings/bindings.cpp

Line 273 in 359b2ba

if (normalize) {

Proposed solution is to move this within the lib

void normalize_vector(float* data, float* norm_array) {
        float norm = 0.0f;
        for (int i = 0; i < dim; i++)
            norm += data[i] * data[i];
        norm = 1.0f / (sqrtf(norm) + 1e-30f);
        for (int i = 0; i < dim; i++)
            norm_array[i] = data[i] * norm;
    }

[yurymalkov]

Hi @DavidGOrtega,
Sure, we can move it. PRs are welcome.
Thanks!

[kiplingw]

I agree. I think it would be a good idea to have this functionality implemented internally, as opposed to being binding specific.