Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ttmenezes/main #1196

Closed
wants to merge 17 commits into from
Closed

Ttmenezes/main #1196

wants to merge 17 commits into from

Conversation

Amnah199
Copy link
Contributor

Proposed Changes:

  • Add kwargs to enable semantic ranking in retrievers.
  • Small fixes

How did you test it?

Ran the tests

Notes for the reviewer

Checklist

ttmenezes and others added 17 commits November 11, 2024 15:16
…epset-ai#1187)

* try pinning pydantic

* retry

* again

* more precise pin

* fix

* better
* Fixes to NvidiaRanker

* Add inits and headers

* More headers

* updates

* Reactivate test

* Fix tests

* Reenable test and add test
* Add schema support for the pgvector document store.

Using the public schema of a PostgreSQL database is an anti-pattern. This change adds support for using a schema other than the public schema to create tables.

* Fix long lines.

* Fix long lines. Remove trailing spaces.

* Fix trailing spaces.

* Fix last trailing space.

* Fix ruff issues.

* Fix trailing space.

* small fixes

---------

Co-authored-by: Stefano Fiorucci <[email protected]>
* Created a model adapter

* Create adapter class and add VertexAPI

* Add chat generator for Anthropic Vertex

* Add tests

* Small fix

* Improve doc_strings

* Make project_id and region mandatory params

* Small fix
@CLAassistant
Copy link

CLAassistant commented Nov 15, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 5 committers have signed the CLA.

✅ ttmenezes
✅ rblst
✅ Amnah199
❌ HaystackBot
❌ sjrl
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Successfully merging this pull request may close these issues.

7 participants