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

Remove while and storages #16

Merged
merged 2 commits into from
Apr 30, 2023
Merged

Remove while and storages #16

merged 2 commits into from
Apr 30, 2023

Conversation

nomisRev
Copy link
Contributor

  • Removes mutable task storage, and flattens result storage into AutoAI.
  • Removes while(true) and replaces with recursion
  • Slightly improve model
  • Remove exceptions, and introduce error models (subsequent PRs)

Having context receivers would drastically improve this design, as everything could be top-level functions and we could use tailrac in ai DSL. This is currently non-problematic since ai will never be such deeply nested stack. It is only as deep as Json retries.

@nomisRev nomisRev requested a review from raulraja April 30, 2023 19:12
|Consider the ultimate objective of your team: ${objective.value}.
|Do not remove any tasks. Return the result as a numbered list, like:
|#. First task
|#. Second task
|Start the task list with number $nextTaskId.""".trimMargin()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure why this nextTaskId was used 🤔
I currently start with TaskId 1 for the original objective, and then increment TaskIds when new generating new tasks through the getNewTasksOrComplete function. The prioritizationAgent should re-use the TaskIds generated by getNewTasksOrComplete but not create new TaskIds, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, we need to revisit the prompts to make them more precise and ensure we are not just generating new tasks. currently seems to fail to complete an objective if it goes into task generation. Seems we are not adding the context properly or in the right format

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I’ll setup logging properly, so we can more easily figure out what’s going on.

@nomisRev nomisRev force-pushed the remove-while-and-storages branch from 0507f18 to a886b71 Compare April 30, 2023 19:25
Copy link
Contributor

@raulraja raulraja left a comment

Choose a reason for hiding this comment

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

thanks @nomisRev !

@raulraja raulraja merged commit ec064c3 into auto-agent Apr 30, 2023
@raulraja raulraja deleted the remove-while-and-storages branch April 30, 2023 21:25
raulraja added a commit that referenced this pull request May 2, 2023
* AI as a language primitive via an auto-agent

* Small clean-up, and fix ktor-client-js issue

* Move around HttpClient dependencies

* Add Simon's PR suggestion to fix bug listing tasks + additional example

* Remove while and storages (#16)

* Flatten structures

* Improve Task and TaskResult model

* Improve logging, and setup example module (#17)

* Improve logging, and setup example module
* Fix package, split logback into TOML, and remove old reference to storage

* Simplify Auto agent by depending on just `ai` function + basic agents draft (#18)

* autonomous agent in terms of `ai` function with self reasoning and agent support.

Extended `ai` function to accept agents

* wikipedia agent, currently failing JS node on import child_process.ExecOptions, io multiplatform solution based on https://github.com/jmfayard/kotlin-cli-starter

* wikipedia agent, currently failing JS node on import child_process.ExecOptions, io multiplatform solution based on https://github.com/jmfayard/kotlin-cli-starter

* Fix build

---------

Co-authored-by: Simon Vergauwen <nomisRev@users.noreply.github.com>

---------

Co-authored-by: Simon Vergauwen <nomisRev@users.noreply.github.com>
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.

None yet

2 participants