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

added Databricks integrations for DSPy LMs/RM configurations #430

Merged
merged 5 commits into from
Feb 25, 2024

Conversation

arnavsinghvi11
Copy link
Collaborator

integrated Databricks connectors for configuring LMs and RM in DSPy.

LMs - adapted from OpenAI Client SDK in Databricks Model Serving endpoints
Supports chat, completions, and embeddings model formats.

RM - adapted from querying Databricks Vector Search Endpoint

@insop
Copy link
Contributor

insop commented Feb 22, 2024

It would be great if you could add LM and RM to the related documents in docs folder.

@arnavsinghvi11
Copy link
Collaborator Author

Thanks @insop. Added the LM docs. The RM docs are being developed in a background docs development (as the current version is missing most of the Retrievers) so will update the Databricks RM with that outside of this PR as well!

@insop
Copy link
Contributor

insop commented Feb 22, 2024

Thanks @insop. Added the LM docs. The RM docs are being developed in a background docs development (as the current version is missing most of the Retrievers) so will update the Databricks RM with that outside of this PR as well!

Thank you @arnavsinghvi11, PR looks good to me.

@okhat okhat merged commit e51b4bd into main Feb 25, 2024
1 check passed
@okhat
Copy link
Collaborator

okhat commented Feb 25, 2024

Thank you so much @arnavsinghvi11 ! And thanks a lot for the review @insop !

arnavsinghvi11 pushed a commit that referenced this pull request Jul 12, 2024
added Databricks integrations for DSPy LMs/RM configurations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants