Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@marklysze
Copy link
Collaborator

@marklysze marklysze commented Jan 7, 2025

Why are these changes needed?

Updates the Cohere client to use V2 of the API, includes Structured Output support.

Related issue number

N/A

Checks

@marklysze
Copy link
Collaborator Author

@Anirudh31415926535, would love your review of this PR with V2

Signed-off-by: Mark Sze <[email protected]>
@marklysze
Copy link
Collaborator Author

@jasonozuzu-cohere, are you happy with the changes made?

@marklysze marklysze disabled auto-merge February 24, 2025 20:24
@marklysze marklysze merged commit bb8b41a into main Feb 24, 2025
17 checks passed
@marklysze marklysze deleted the coherev2 branch February 24, 2025 20:24
@codecov
Copy link

codecov bot commented Feb 24, 2025

Codecov Report

Attention: Patch coverage is 9.83607% with 110 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
autogen/oai/cohere.py 9.83% 101 Missing and 9 partials ⚠️

❗ There is a different number of reports uploaded between BASE (1792140) and HEAD (63bea7e). Click for more details.

HEAD has 1286 uploads less than BASE
Flag BASE (1792140) HEAD (63bea7e)
3.10 99 0
ubuntu-latest 145 1
falkordb 2 0
3.11 66 1
3.13 88 0
lmm 4 0
teachable 4 0
core-llm 4 0
3.9 80 0
gemini 15 0
anthropic 16 0
core-without-llm 14 1
autobuild 1 0
macos-latest 108 0
integration 11 0
browser-use 6 0
3.12 39 0
neo4j 2 0
windows-latest 119 0
crawl4ai 13 0
deepseek 1 0
interop 13 0
agent-eval 1 0
retrievechat 15 0
retrievechat-qdrant 14 0
gpt-assistant-agent 3 0
openai 1 0
cerebras 14 0
optional-deps 141 0
retrievechat-couchbase 3 0
retrievechat-mongodb 10 0
retrievechat-pgvector 10 0
commsagent-discord 9 0
llama-index-agent 3 0
commsagent-slack 9 0
commsagent-telegram 9 0
long-context 3 0
jupyter-executor 9 0
websurfer 15 0
cohere 15 0
graph-rag-falkor-db 6 0
twilio 9 0
groq 14 0
mistral 14 0
ollama 14 0
together 14 0
bedrock 14 0
swarm 14 0
reasoning 14 0
docs 6 0
interop-langchain 9 0
interop-crewai 9 0
interop-pydantic-ai 9 0
websockets 9 0
Files with missing lines Coverage Δ
autogen/oai/cohere.py 28.57% <9.83%> (-7.75%) ⬇️

... and 62 files with indirect coverage changes

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.

5 participants