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

Add prompt rewriting for multiple messages inReasoningAgent #758

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

giorgossideris
Copy link
Collaborator

@giorgossideris giorgossideris commented Feb 2, 2025

Why are these changes needed?

Related issue number

Closes #755

Checks

@CLAassistant
Copy link

CLAassistant commented Feb 2, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

codecov bot commented Feb 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
3.10 58.43% <ø> (?)
3.11 29.28% <ø> (?)
3.12 27.54% <ø> (?)
3.13 41.61% <ø> (?)
3.9 40.06% <ø> (?)
anthropic 16.49% <ø> (?)
bedrock 16.87% <ø> (?)
cerebras 16.58% <ø> (?)
cohere 16.63% <ø> (?)
docs 28.85% <ø> (?)
gemini 16.57% <ø> (?)
graph-rag-falkor-db 28.28% <ø> (?)
groq 16.60% <ø> (?)
interop 29.73% <ø> (?)
jupyter-executor 31.00% <ø> (?)
lmm 19.46% <ø> (?)
long-context 15.73% <ø> (?)
macos-latest 58.49% <ø> (?)
mistral 16.30% <ø> (?)
ollama 16.98% <ø> (?)
retrievechat 29.85% <ø> (?)
retrievechat-mongodb 29.84% <ø> (?)
retrievechat-pgvector 29.84% <ø> (?)
retrievechat-qdrant 29.79% <ø> (?)
teachable 15.90% <ø> (?)
together 16.63% <ø> (?)
twilio 28.49% <ø> (?)
ubuntu-latest 58.52% <ø> (?)
unittests ?
websurfer 18.87% <ø> (?)
windows-latest 56.12% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 74 files with indirect coverage changes

@giorgossideris giorgossideris requested a review from Hk669 February 3, 2025 11:38
@Hk669
Copy link
Collaborator

Hk669 commented Feb 4, 2025

@giorgossideris hey, added a comment in the issue. can you please check and give us any suggestions on that? thanks.

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.

[Bug]: ReasoningAgent's _process_prompt doesn't handle multiple messages.
4 participants