Pgvector fixes

[edlouth]

Fixes for #659

@andreped could you take a look?

[edlouth]

All of @andreped's changes are now merged into this PR and I think it is ready to merge.

[andreped]

All of @andreped's changes are now merged into this PR and I think it is ready to merge.

@edlouth Did you manage to run the pgvector tests? If not, could you do so, to make sure this PR is good?

pytest -v tests/test_pgvector.py

[edlouth]

All of @andreped's changes are now merged into this PR and I think it is ready to merge.

@edlouth Did you manage to run the pgvector tests? If not, could you do so, to make sure this PR is good?

pytest -v tests/test_pgvector.py

The tests are passing for me, I made a small change to bring in the database port from an environment variable.

[andreped]

The tests are passing for me, I made a small change to bring in the database port from an environment variable.

Great! Then I think PR is good!

Still requires approval from @zainhoda before merge though, but great job, @edlouth! :]

[andreped]

LGTM!

[zainhoda]

I commented out the tests because I'm getting a bunch of dependency issues trying to run them. We can add them back later

[andreped]

No worries, @zainhoda. As we already talked about, I can make a PR to add these tests in a more proper way, such that they will not conflict with the tests you are already running on your end (potentially not in GitHub Actions).

You should be able to test individual components easily without the full blown tests.