-
Notifications
You must be signed in to change notification settings - Fork 55
refactor: params to plan prompt #709
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
base: main
Are you sure you want to change the base?
Conversation
a5da059 to
560861d
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #709 +/- ##
==========================================
+ Coverage 65.34% 67.95% +2.61%
==========================================
Files 78 78
Lines 3388 3405 +17
==========================================
+ Hits 2214 2314 +100
+ Misses 1174 1091 -83 ☔ View full report in Codecov by Sentry. |
|
Hello @deepkarkada, thanks for your interest in contributing! Based on your interest for agentic programming, this PR relates to the long-horizon planning agent and could use your feedback. A good starting point for understanding this PR is to look into #679 which describes some initial design decisions of the agent. One way to get hands-on experience is to set up your dev environment to run the script from PR #679. To do that, you'll need to set up RAI. Instructions can be found at https://robotecai.github.io/rai/setup/install/. Please feel free to ping technical-support or reach out to me if you run into any issues setting up your machine. Any of us will be happy to walk you through the steps. As for timeline, since this is your first PR, I wouldn't stress about it. As long as you get started, the overall goal is achieved. |
d13face to
46de30b
Compare
|
brought back previous arg name 3409d0a I m not sure if I should leave this name |
|
Hi @jmatejcz, Julia added me to the PR to help test new PlanPrompts. I have a few clarification questions about the agent design. From my understanding of PR #679 : the megamind agent now combines planner, replanner and supervisor. Essentially, there is no planning as such, the execution is step-by-step where given the objective, megamind comes up with first step, second step etc and binds it to the executor with the right tools. I guess my confusion/lack of context is around configuring the planning prompt. Is it configuring the subtask for each step? Is it possible to do so without knowing what step is generated. If there is a PR around more context to the PlanPrompts, please feel free to share, I would definitely be interested to learn more. |
About the confusion with
Idk if you read comments by me and @Juliaj under PR #679 - this is origin of the idea |
Purpose
Proposed Changes
PlanPrompts dataclass
Issues
Testing
Test it out with some random prompts in file
rai_bench/examples/tool_calling_custom_agent.pyYou should see difference in user prompt