Skip to content

Conversation

@BeatrixCohere
Copy link
Contributor

No description provided.

@arnavsinghvi11
Copy link
Collaborator

Hi @BeatrixCohere , why is the client_name=dspy needed? I have not used Cohere myself, but curious where this client_name is getting used.

@BeatrixCohere
Copy link
Contributor Author

Hi @BeatrixCohere , why is the client_name=dspy needed? I have not used Cohere myself, but curious where this client_name is getting used.

Hi, thanks for reviewing. It aids in debugging issues with the dspy integration with Cohere.

@jalammar
Copy link
Contributor

@arnavsinghvi11 This change would help prioritize DSPy at Cohere and contribute to how the models may work in the future to support its use cases.

@arnavsinghvi11 arnavsinghvi11 merged commit 7484f26 into stanfordnlp:main May 15, 2024
@arnavsinghvi11
Copy link
Collaborator

Merged. thanks @BeatrixCohere !

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