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

Dev -> Master: Codex RC1 for alpha testers #73

Merged
merged 69 commits into from
Mar 13, 2024
Merged

Dev -> Master: Codex RC1 for alpha testers #73

merged 69 commits into from
Mar 13, 2024

Conversation

Swiftyos
Copy link
Contributor

@Swiftyos Swiftyos commented Mar 5, 2024

Draft Dev to Master PR, whilst we are developing the next batch of features.

Goal:

System is working without errors for all examples we can think of that do not require a database.

Whats expected to be included:

  • Flatter Functions database structure @majdyz
  • Updates Requirements gens param structure @ntindle
  • End-to-end benchmark running @Swiftyos
  • Ensure code generation uses pydantic objects correctly
  • Bug fixes required before merge AGPT-483

Release Plan:

Target date: 6th March 2024

  • Test benchmarks locally (manual, see last comment)
  • Merge into master
  • Deploy to dev env (automatic)
  • Test with discord bot (discord)

majdyz and others added 24 commits March 1, 2024 13:32
…r-generated-function' of github.com:Significant-Gravitas/codex into zamilmajdy/agpt-433-argument-and-return-type-parsing-for-generated-function
…-return-type-parsing-for-generated-function' into zamilmajdy/agpt-433-argument-and-return-type-parsing-for-generated-function

# Conflicts:
#	codex/develop/agent.py
Propagate DB schema changes: Function <-> CompiledRouteSpec (#65)

---------

Co-authored-by: majdyz <[email protected]>
Co-authored-by: Swifty <[email protected]>
Co-authored-by: majdyz <[email protected]>
@Swiftyos Swiftyos marked this pull request as ready for review March 5, 2024 13:15
@Swiftyos Swiftyos marked this pull request as draft March 5, 2024 13:15
@Swiftyos
Copy link
Contributor Author

Swiftyos commented Mar 6, 2024

If we are ready to merge before I'm back @majdyz or @ntindle needs to take on the responsibility of testing the branch before merging

# Running the benchmarks so you can check this again:

./scripts/db.sh
./run serve

# Then in another terminal to run all the benchmarks run this command

./run benchmark

# If you would just like to run a single example:

./run example

ntindle and others added 21 commits March 6, 2024 21:28
* feat: make the name match

* feat: database becomes optional

* Update agent.py
* feat: make the name match

* feat: database becomes optional

* feat: initial model changes

* feat: add prompts

* feat: more prompts

* fix: nits

* fix: wrong type 😢

* feat: improved prompts
[AGPT-489] Fix function fetching to always include RelatedTypes

---------

Co-authored-by: majdyz <[email protected]>
* feat: endpoint updates

* fix: ambiguous param
Co-authored-by: majdyz <[email protected]>
…ng to LLM to guess wrong signature (#97)

[AGPT-485] Fix missing pydantic object during requirement-step; leading to LLM to guess wrong signature

---------

Co-authored-by: majdyz <[email protected]>
* Added prisma client to server code

* Add DatabaseSchema to INCLUDE_FUNC

* Update functions to be async and add database schema documentation

* Add DatabaseSchema and DatabaseTables to ApiRouteSpec

* Updated dev prompts for db

* Update profile management descriptions

* Update route function definition to be async

* Remove logging of database schema

* Add Prisma schema file generation

* Refactor benchmark and agent modules

* Refactor code and comment out debug statements

* formatting

* Add available_functions to develop_route

* Update prisma import and method calls

* loading all related tables

* Refactor create_prisma_scheama_file function to handle cases with no database schema

* Fix async function call in packager.py

* Fix database table query in packager.py

* Fix database access and unnecessary imports

* review comments
* Added prisma client to server code

* Add DatabaseSchema to INCLUDE_FUNC

* Update functions to be async and add database schema documentation

* Add DatabaseSchema and DatabaseTables to ApiRouteSpec

* Updated dev prompts for db

* Update profile management descriptions

* Update route function definition to be async

* Remove logging of database schema

* Add Prisma schema file generation

* Refactor benchmark and agent modules

* Refactor code and comment out debug statements

* formatting

* Add available_functions to develop_route

* Update prisma import and method calls

* loading all related tables

* Refactor create_prisma_scheama_file function to handle cases with no database schema

* Fix async function call in packager.py

* Fix database table query in packager.py

* Fix database access and unnecessary imports

* feat: add enums to the database gen

* fix: formatting

* Updated to new schema

* update hardcoded requirements

* fix: objectType generation without type bool

* Add database enum import statement

* removed unused var as failing lint check

---------

Co-authored-by: SwiftyOS <[email protected]>
* fix: more linting issues

* Remove error log for async functions

* Fix return_type default value in FunctionDef class

* Added another validation check

---------

Co-authored-by: Swifty <[email protected]>
@Swiftyos Swiftyos marked this pull request as ready for review March 13, 2024 14:46
@Swiftyos Swiftyos requested review from ntindle and majdyz March 13, 2024 14:47
@Swiftyos Swiftyos changed the title Dev -> Master Dev -> Master: Codex RC1 for alpha testers Mar 13, 2024
@@ -0,0 +1,623 @@
import logging
from datetime import datetime
Copy link
Member

Choose a reason for hiding this comment

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

Is this from Cloud Services?

@Swiftyos Swiftyos merged commit e79d563 into main Mar 13, 2024
3 checks passed
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.

3 participants