enhancement: add supervisor agent for routing query#42
Conversation
george-lhj
changed the title
|
addressed comments @maximilien |
george-lhj
force-pushed
the
fix-supervisor-timeout-and-routing
branch
from
0989cc5 to
8a3a034
Compare
|
@maximilien could you review this one? you requested changes to break apart into smaller functions and i finished the refactor while you were on vacation |
maximilien
left a comment
maximilien
left a comment
There was a problem hiding this comment.
Also separate the APIs from the supervisor agent code. The API should be just calling the agent and other code as well as dealing with API issues (check for errors in input etc).
Sorry about late review. Please tag me and will try to respond within 1/2 day or so.
george-lhj
force-pushed
the
fix-supervisor-timeout-and-routing
branch
from
8a3a034 to
9210efc
Compare
|
LMK when ready for final review |
|
|
ok i think it is ready for review @maximilien |
george-lhj
force-pushed
the
fix-supervisor-timeout-and-routing
branch
from
b320de5 to
18afb03
Compare
|
finished @maximilien added feature to auto show the agents once generated |
maximilien
left a comment
maximilien
left a comment
There was a problem hiding this comment.
LGTM based on demo and quick review of code ... goal is to move fast and get users :)
2 participants
fixes #40
adds a supervisor agent as the first layer so that user only needs to input command, then the agent routes it to the correect endpoint