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

Update docs quick overview #988

Merged

Conversation

kristiankyvik
Copy link
Collaborator

@kristiankyvik kristiankyvik commented Jan 23, 2024

My attempt here was to update the Quick overview section of the docs, since it seems to be out of date.

Here are the summary of the changes:

  • Attempted to update the core components to reflect new structure
  • Removed references to old classes and functions and updated the file paths
  • Added in-page anchors when required

I also documented a command to run the docs with live reload, which will make it easier to work with them locally.

### 1. AI Class (`gpt_engineer/ai.py`)
The AI class is the main interface to the GPT-4 model. It provides methods to start a conversation with the model, continue an existing conversation, and format system and user messages.

(ai_class)=
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this some markdown extension I'm not aware of? Never seen before 😮

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Me neither! Apparently, its Sphinx way of doing cross-references in the MyST Markdown dialect (see docs).

@ATheorell
Copy link
Collaborator

Maybe outside the scope of what you want to do in this PR @kristiankyvik, but feel free to cut down A LOT on the static documentation files (the ones not generated from docstrings). Chances are that they are outdated and it doesn't seem like we are managing to keep non-generated doc files up-to-date.

@kristiankyvik
Copy link
Collaborator Author

kristiankyvik commented Jan 24, 2024

Hi @ATheorell,
I think what you suggest makes a lot of sense. I am happy to take a look at the automatically generated docs (API Reference) next. I will open an issue now to suggest what we can improve.

Could it make sense to merge this PR in the meantime so that at least the main overview is up-to-date?

@TheoMcCabe TheoMcCabe merged commit a25e7a6 into gpt-engineer-org:main Jan 30, 2024
3 checks passed
@TheoMcCabe
Copy link
Collaborator

TheoMcCabe commented Jan 30, 2024

Looks good! Thank you @kristiankyvik . Yes would be great if you had time to look at the auto generated docs too

Edit -looks like you already are looking at that. Nice one

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.

4 participants