Skip to content

Unsupported lang fix #745

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 3 commits into from
Nov 10, 2023

Conversation

CyanideByte
Copy link
Contributor

Describe the changes you have made:

When unsupported language is encountered, it continues the response so the model can try workarounds and continue with the remaining steps instead of immediately stopping. Also added a small check so that if the model tries to read out the code a 2nd time it doesnt loop.

Reference any relevant issue (Fixes #000)

  • I have performed a self-review of my code:

I have tested the code on the following OS:

  • Windows
  • MacOS
  • Linux

AI Language Model (if applicable)

  • GPT4
  • GPT3
  • Llama 7B
  • Llama 13B
  • Llama 34B
  • Huggingface model (Please specify which one)

@Notnaton
Copy link
Collaborator

@ericrallen I closed my PR in favor of this one. It is much simpler and doesn't require my hacky solution :)
I have tested this PR, works as intended on my machine.
Great work @CyanideByte

While im in this file, heres a small fix for eb8's issue on discord.
KeyError: ' python'
The issue was caused by the llm responding with a token starting with a space in front of the language.
@KillianLucas
Copy link
Collaborator

Very clean patch. Thanks @CyanideByte for fixing this! And thanks @Notnaton for your PR and for testing this one. Merging.

@KillianLucas KillianLucas merged commit ae97d9a into openinterpreter:main Nov 10, 2023
@CyanideByte CyanideByte deleted the unsupported-lang-fix branch November 10, 2023 23:05
joshuavial pushed a commit to joshuavial/open-interpreter that referenced this pull request Nov 16, 2023
…ang-fix

Unsupported lang fix

Former-commit-id: ae97d9a
Former-commit-id: cec21b19103eeb20a540eba55cc114865404b6af
Former-commit-id: 80923ac9e44f4c0402273ae7a91639c85b08e0c7 [formerly 45e8c5388fd0e3beae39d5bf61718ff9deefe8ed]
Former-commit-id: 52ca83d38df5fb754917838676b97a775b801d85
joshuavial pushed a commit to joshuavial/open-interpreter that referenced this pull request Nov 16, 2023
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