Skip to content

Add doc strings to all computer API functions for the LLM to use in the skill library #1011

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

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

tyfiero
Copy link
Contributor

@tyfiero tyfiero commented Feb 13, 2024

Describe the changes you have made:

Added doc strings to all computer API functions for use by the skill library. These short doc strings will be used to semantically search for the most relevant computer API function to use.

Note: I removed "Args" inside of the existing doc strings, it sounds like we will programmatically inject that information into the skill library instead of writing it all out, as far as I understand. Happy to add them back or add Args to all doc strings if necessary :)

Reference any relevant issues (e.g. "Fixes #000"):

Pre-Submission Checklist (optional but appreciated):

  • I have included relevant documentation updates (stored in /docs)
  • I have read docs/CONTRIBUTING.md
  • I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux

@KillianLucas
Copy link
Collaborator

Awesome, thanks Ty! Super helpful to the 01 + OI projects and to many language models in the future lol. Merging!

@KillianLucas KillianLucas merged commit 6da643e into openinterpreter:main Feb 14, 2024
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.

2 participants