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

Summarize reasoning engine response and misc #19

Merged
merged 5 commits into from
Oct 28, 2024
Merged

Conversation

ashish-spext
Copy link
Contributor

  • Add additional prompt to summarize the run (in case of agents).
  • Handle edge case of closing process on interrupt in make file.
  • Agents
    • Stream Agent: send stream.
    • Pricing Agent: send llm response for context.

Handle edge case of closing process on interrupt in make file.
- Agents
  - Stream Agent: send stream.
  - Pricing Agent: send llm response for context.
@ashish-spext ashish-spext changed the title Summarize reasoning engine response Summarize reasoning engine response and misc Oct 25, 2024
Copy link
Member

@codeAshu codeAshu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes looks good. Are they ready to merge in main?

…deo summary

Download respond with info that response was not displayed
RE add condition to display response in summary if not already done by agent
Remove it in case of agent is selected again in iteration (think upload -> summarize)
Put both add and remove in a method
Add reasoning_engine as agent name for RE summary
@ashish-spext ashish-spext merged commit ed86d06 into main Oct 28, 2024
@ashish-spext ashish-spext deleted the re-summary branch October 28, 2024 11:59
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.

2 participants