-
Notifications
You must be signed in to change notification settings - Fork 297
Post-review modifications to Planner agent docs #1301
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
Post-review modifications to Planner agent docs #1301
Conversation
antoniibelyshev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! I left a couple of small comments about the usage of fun main(): it 's better to include additional value definitions like val agent = ...in the main. The changes look good
| <!--- SUFFIX | ||
| } | ||
| --> | ||
| ```kotlin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suspend fun main() should start here
| val isPublished: Boolean = false | ||
| ) | ||
|
|
||
| // Create GOAP planner with LLM-powered actions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the main should start here
1393a0a to
ff469ea
Compare
ff469ea to
95fca66
Compare
Review and suggest changes to Planner agents docs.
Type of the changes
Checklist
developas the base branchAdditional steps for pull requests adding a new feature