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

Readme shorted. Docs referenced #465

Merged
merged 2 commits into from
Jan 10, 2024
Merged

Readme shorted. Docs referenced #465

merged 2 commits into from
Jan 10, 2024

Conversation

jakethekoenig
Copy link
Member

Pull Request Checklist

  • Documentation has been updated, or this change doesn't require that

@@ -173,4 +89,4 @@ The goal for Mentat is to become the best way to write code. Some big improvemen

If this is something you'd like to work on, jump right in! And if you want to join the team to work on this full time, message [@bio_bootloader](https://twitter.com/bio_bootloader) on twitter.

To find something specific to work on, take a look at [open issues](https://github.com/AbanteAI/mentat/issues) and/or check this Github Project: [Mentat Development](https://github.com/users/biobootloader/projects/2)
To find something specific to work on, take a look at [open issues](https://github.com/AbanteAI/mentat/issues).
Copy link
Member Author

Choose a reason for hiding this comment

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

It doesn't seem like we really use the project so I thought it was better to remove the link.

README.md Outdated
## Selecting which LLM Model to use

We highly recommend using the default model, `gpt-4-1106-preview`, as it performs vastly better than any other model benchmarked so far. However, if you wish to use a different model, jump [here](#alternative-models).

### Add your OpenAI API Key

You'll need to have API access to GPT-4 to run Mentat. There are a few options to provide Mentat with your OpenAI API key:
Copy link
Member

Choose a reason for hiding this comment

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

Maybe this should be changed; everybody has gpt-4 access now, and it isn't even necessary anymore.

@@ -70,66 +67,6 @@ You'll need to have API access to GPT-4 to run Mentat. There are a few options t
2. Run `export OPENAI_API_KEY=<your key here>` prior to running Mentat
3. Place the previous command in your `.bashrc` or `.zshrc` to export your key on every terminal startup

### Azure OpenAI
Copy link
Member

Choose a reason for hiding this comment

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

Is any of this information anywhere else? I think it's important to keep somewhere

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Could we link here here? I think it's important that it's easy for people to find the docs for alternative models

@@ -1,5 +1,6 @@
[![Twitter Follow](https://img.shields.io/twitter/follow/bio_bootloader?style=social)](https://twitter.com/bio_bootloader)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
[![Twitter Follow](https://img.shields.io/twitter/follow/bio_bootloader?style=social)](https://twitter.com/bio_bootloader)
[![Twitter Follow](https://img.shields.io/twitter/follow/AbanteAI?style=social)](https://twitter.com/AbanteAI)

@granawkins granawkins merged commit 956d961 into main Jan 10, 2024
16 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.

None yet

4 participants