-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Rename methods and tweak API #738
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #738 +/- ##
==========================================
- Coverage 61.22% 61.18% -0.04%
==========================================
Files 101 101
Lines 12352 12371 +19
==========================================
+ Hits 7562 7569 +7
- Misses 4790 4802 +12 ☔ View full report in Codecov by Sentry. |
1137073
to
75eefa0
Compare
Looking good, but thinking about it again I'm not sure I love |
AgentStepResults did seem superfluous besides making the return type the same so I like that idea of merging those two methods |
Maybe render_output can be controlled by the LLM too, e.g. prompt
|
answer
->step_through
Agent().invoke
->respond
step.stream
instep_through
, andinterface.stream
inrespond
step_through
inAgentStepResults
parameterizedAgentChainLink
parameterized